Update the CaretBrowsingMode toolbar-height if the toolbarDensity…

… preference changes (PR 18786 follow-up) Otherwise the isVisible-calculations may not work correctly.
mozilla · Oct 1, 2024 · e9a77f5 · e9a77f5
1 parent f2a132f
commit e9a77f5
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/web/app.js b/web/app.js
@@ -876,6 +876,7 @@ const PDFViewerApplication = {
 
   moveCaret(isUp, select) {
     this._caretBrowsing ||= new CaretBrowsingMode(
+      this.eventBus,
       this.appConfig.mainContainer,
       this.appConfig.viewerContainer,
       this.appConfig.toolbar?.container

diff --git a/web/caret_browsing.js b/web/caret_browsing.js
@@ -23,10 +23,20 @@ class CaretBrowsingMode {
 
   #viewerContainer;
 
-  constructor(mainContainer, viewerContainer, toolbarContainer) {
+  constructor(eventBus, mainContainer, viewerContainer, toolbarContainer) {
     this.#mainContainer = mainContainer;
     this.#viewerContainer = viewerContainer;
-    this.#toolBarHeight = toolbarContainer?.getBoundingClientRect().height ?? 0;
+
+    const computeToolBarHeight = () => {
+      this.#toolBarHeight =
+        toolbarContainer?.getBoundingClientRect().height ?? 0;
+    };
+    computeToolBarHeight();
+
+    eventBus._on("toolbardensity", async () => {
+      await Promise.resolve(); // Ensure that the UI has updated.
+      computeToolBarHeight();
+    });
   }
 
   /**