[Editor] Don't show the alt-text button when the alt-text dialog is v…

…isible This way, the button doens't cover the image.
mozilla · Sep 25, 2023 · 72e2a02 · 72e2a02
1 parent 57d196e
commit 72e2a02
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/src/display/editor/editor.js b/src/display/editor/editor.js
@@ -836,6 +836,7 @@ class AnnotationEditor {
     altText.addEventListener(
       "click",
       event => {
+        this.#altTextButton.hidden = true;
         event.preventDefault();
         this._uiManager.editAltText(this);
       },
@@ -919,6 +920,10 @@ class AnnotationEditor {
     }
   }
 
+  altTextFinish() {
+    this.#altTextButton.hidden = false;
+  }
+
   getClientDimensions() {
     return this.div.getBoundingClientRect();
   }

diff --git a/web/alt_text_manager.js b/web/alt_text_manager.js
@@ -264,6 +264,7 @@ class AltTextManager {
     this.#removeOnClickListeners();
     this.#uiManager?.addEditListeners();
     this.#eventBus._off("resize", this.#boundSetPosition);
+    this.#currentEditor.altTextFinish();
     this.#currentEditor = null;
     this.#uiManager = null;
   }