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

Protect a few additional DOM element accesses in the viewer (PR 15831 follow-up)

Browse Source 
A couple of cases that I missed during review, for code-paths that don't run by default in the viewer.
This commit is contained in:
Jonas Jenwald 2022-12-15 18:48:10 +01:00
parent 826c358b3a
commit 4bd66a2150
1 changed files with 9 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
web/app.js
View File

@ -532,7 +532,7 @@ const PDFViewerApplication = {
pdfLinkService.setHistory(this.pdfHistory); pdfLinkService.setHistory(this.pdfHistory);
} }
if (appConfig.findBar && !this.supportsIntegratedFind) { if (!this.supportsIntegratedFind && appConfig.findBar) {
this.findBar = new PDFFindBar(appConfig.findBar, eventBus, this.l10n); this.findBar = new PDFFindBar(appConfig.findBar, eventBus, this.l10n);
} }
@ -543,11 +543,8 @@ const PDFViewerApplication = {
eventBus eventBus
); );
} else { } else {
for (const element of [ for (const id of ["editorModeButtons", "editorModeSeparator"]) {
document.getElementById("editorModeButtons"), document.getElementById(id)?.classList.add("hidden");
document.getElementById("editorModeSeparator"),
]) {
element.hidden = true;
} }
} }
} }
@ -792,16 +789,13 @@ const PDFViewerApplication = {
* @private * @private
*/ */
_hideViewBookmark() { _hideViewBookmark() {
const { viewBookmarkButton, presentationModeButton } = const { secondaryToolbar } = this.appConfig;
this.appConfig.secondaryToolbar;
// URL does not reflect proper document location - hiding some buttons. // URL does not reflect proper document location - hiding some buttons.
viewBookmarkButton.hidden = true; secondaryToolbar?.viewBookmarkButton.classList.add("hidden");
// Avoid displaying multiple consecutive separators in the secondaryToolbar. // Avoid displaying multiple consecutive separators in the secondaryToolbar.
if (presentationModeButton.hidden) { if (secondaryToolbar?.presentationModeButton.classList.contains("hidden")) {
const element = document.getElementById("viewBookmarkSeparator"); document.getElementById("viewBookmarkSeparator")?.classList.add("hidden");
element.hidden = true;
} }
}, },
@ -2175,8 +2169,8 @@ function webViewerInitialized() {
appConfig.secondaryToolbar?.printButton.classList.add("hidden"); appConfig.secondaryToolbar?.printButton.classList.add("hidden");
} }
if (appConfig.secondaryToolbar && !PDFViewerApplication.supportsFullscreen) { if (!PDFViewerApplication.supportsFullscreen) {
appConfig.secondaryToolbar.presentationModeButton.hidden = true; appConfig.secondaryToolbar?.presentationModeButton.classList.add("hidden");
} }
if (PDFViewerApplication.supportsIntegratedFind) { if (PDFViewerApplication.supportsIntegratedFind) {