Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't remove the loading icon from the DOM when a page is resetted

Browse Source
This commit is contained in:
Calixte Denizet 2022-12-14 18:13:09 +01:00
parent d90e62e806
commit 20037e9919
1 changed files with 13 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
web/pdf_page_view.js
View File

@ -429,6 +429,7 @@ class PDFPageView {
case annotationEditorLayerNode: case annotationEditorLayerNode:
case xfaLayerNode: case xfaLayerNode:
case textLayerNode: case textLayerNode:
case this.loadingIconDiv:
continue; continue;
} }
node.remove(); node.remove();
@ -472,8 +473,18 @@ class PDFPageView {
delete this.svg; delete this.svg;
} }
this.loadingIconDiv = document.createElement("div"); if (!this.loadingIconDiv) {
this.loadingIconDiv.className = "loadingIcon notVisible"; this.loadingIconDiv = document.createElement("div");
this.loadingIconDiv.className = "loadingIcon notVisible";
this.loadingIconDiv.setAttribute("role", "img");
this.l10n.get("loading").then(msg => {
this.loadingIconDiv?.setAttribute("aria-label", msg);
});
div.append(this.loadingIconDiv);
} else {
this.toggleLoadingIconSpinner();
}
if ( if (
(typeof PDFJSDev === "undefined" || (typeof PDFJSDev === "undefined" ||
PDFJSDev.test("!PRODUCTION || GENERIC")) && PDFJSDev.test("!PRODUCTION || GENERIC")) &&
@ -481,11 +492,6 @@ class PDFPageView {
) { ) {
this.toggleLoadingIconSpinner(/* viewVisible = */ true); this.toggleLoadingIconSpinner(/* viewVisible = */ true);
} }
this.loadingIconDiv.setAttribute("role", "img");
this.l10n.get("loading").then(msg => {
this.loadingIconDiv?.setAttribute("aria-label", msg);
});
div.append(this.loadingIconDiv);
} }
update({ scale = 0, rotation = null, optionalContentConfigPromise = null }) { update({ scale = 0, rotation = null, optionalContentConfigPromise = null }) {