Simplify the PDFSidebar.#forceRendering method

The fallback code-path has never really been used, since the `PDFSidebar` is only used in the default viewer (and has never been exposed in e.g. the COMPONENTS-build).
This commit is contained in:
Jonas Jenwald 2023-05-02 14:16:18 +02:00
parent 4931f293eb
commit 9dcc50402e

View File

@ -156,7 +156,7 @@ class PDFSidebar {
*/
switchView(view, forceOpen = false) {
const isViewChanged = view !== this.active;
let shouldForceRendering = false;
let forceRendering = false;
switch (view) {
case SidebarView.NONE:
@ -166,7 +166,7 @@ class PDFSidebar {
return; // Closing will trigger rendering and dispatch the event.
case SidebarView.THUMBS:
if (this.isOpen && isViewChanged) {
shouldForceRendering = true;
forceRendering = true;
}
break;
case SidebarView.OUTLINE:
@ -224,9 +224,9 @@ class PDFSidebar {
this.open();
return; // Opening will trigger rendering and dispatch the event.
}
if (shouldForceRendering) {
if (forceRendering) {
this.#updateThumbnailViewer();
this.#forceRendering();
this.onToggled();
}
if (isViewChanged) {
this.#dispatchEvent();
@ -246,7 +246,7 @@ class PDFSidebar {
if (this.active === SidebarView.THUMBS) {
this.#updateThumbnailViewer();
}
this.#forceRendering();
this.onToggled();
this.#dispatchEvent();
this.#hideUINotification();
@ -263,7 +263,7 @@ class PDFSidebar {
this.outerContainer.classList.add("sidebarMoving");
this.outerContainer.classList.remove("sidebarOpen");
this.#forceRendering();
this.onToggled();
this.#dispatchEvent();
}
@ -286,16 +286,6 @@ class PDFSidebar {
});
}
#forceRendering() {
if (this.onToggled) {
this.onToggled();
} else {
// Fallback
this.pdfViewer.forceRendering();
this.pdfThumbnailViewer.forceRendering();
}
}
#updateThumbnailViewer() {
const { pdfViewer, pdfThumbnailViewer } = this;