[Editing] Disable buttons until the first page is rendered

mozilla · Aug 9, 2022 · ae3742a · ae3742a
1 parent 589f72e
commit ae3742a
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 9 deletions.
diff --git a/web/base_viewer.js b/web/base_viewer.js
@@ -727,12 +727,6 @@ class BaseViewer {
           if (isPureXfa) {
             console.warn("Warning: XFA-editing is not implemented.");
           } else if (isValidAnnotationEditorMode(mode)) {
-            // Ensure that the Editor buttons, in the toolbar, are updated.
-            this.eventBus.dispatch("annotationeditormodechanged", {
-              source: this,
-              mode,
-            });
-
             this.#annotationEditorUIManager = new AnnotationEditorUIManager(
               this.container,
               this.eventBus
@@ -817,6 +811,14 @@ class BaseViewer {
             this._scriptingManager.setDocument(pdfDocument); // Enable scripting.
           }
 
+          if (this.#annotationEditorUIManager) {
+            // Ensure that the Editor buttons, in the toolbar, are updated.
+            this.eventBus.dispatch("annotationeditormodechanged", {
+              source: this,
+              mode: this.#annotationEditorMode,
+            });
+          }
+
           // In addition to 'disableAutoFetch' being set, also attempt to reduce
           // resource usage when loading *very* long/large documents.
           if (

diff --git a/web/toolbar.js b/web/toolbar.js
@@ -244,15 +244,17 @@ class Toolbar {
       }
     };
     this.eventBus._on("annotationeditormodechanged", editorModeChanged);
-
-    this.eventBus._on("toolbarreset", evt => {
+    const reset = evt => {
       if (evt.source === this) {
         editorModeChanged(
           { mode: AnnotationEditorType.NONE },
           /* disableButtons = */ true
         );
       }
-    });
+    };
+
+    this.eventBus._on("toolbarreset", reset);
+    reset({ source: this });
   }
 
   _updateUIState(resetNumPages = false) {