Remove focus from the toggleButton when closing the sidebar with a mo…

…use (issue 17361)
mozilla · Dec 6, 2023 · 2abdc22 · 2abdc22
1 parent 7e64f82
commit 2abdc22
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/web/pdf_sidebar.js b/web/pdf_sidebar.js
@@ -264,7 +264,7 @@ class PDFSidebar {
     this.#hideUINotification();
   }
 
-  close() {
+  close(evt = null) {
     if (!this.isOpen) {
       return;
     }
@@ -276,11 +276,16 @@ class PDFSidebar {
 
     this.onToggled();
     this.#dispatchEvent();
+
+    if (evt?.detail > 0) {
+      // Remove focus from toggleButton if it was clicked (see issue 17361).
+      this.toggleButton.blur();
+    }
   }
 
-  toggle() {
+  toggle(evt = null) {
     if (this.isOpen) {
-      this.close();
+      this.close(evt);
     } else {
       this.open();
     }
@@ -334,8 +339,8 @@ class PDFSidebar {
       }
     });
 
-    this.toggleButton.addEventListener("click", () => {
-      this.toggle();
+    this.toggleButton.addEventListener("click", evt => {
+      this.toggle(evt);
     });
 
     // Buttons for switching views.