From 6468e1d0cdcd040a2bcf5acc010b58ad1790e235 Mon Sep 17 00:00:00 2001
From: Jonas Jenwald <jonas.jenwald@gmail.com>
Date: Thu, 20 May 2021 15:17:01 +0200
Subject: [PATCH] Remove focus from the zoom dropdown, when a mouse is used
 (bug 1300525, issue 4923)

This patch fixes the referenced bugs/issues, in a way that won't interfere with keyboard users, assuming that we actually want to fix these old bugs/issues. (If not, we should close them as WONTFIX.)
---
 web/toolbar.js | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/web/toolbar.js b/web/toolbar.js
index 6b6353dbc..102834442 100644
--- a/web/toolbar.js
+++ b/web/toolbar.js
@@ -155,6 +155,19 @@ class Toolbar {
         value: this.value,
       });
     });
+    // Here we depend on browsers dispatching the "click" event *after* the
+    // "change" event, when the <select>-element changes.
+    scaleSelect.addEventListener("click", function (evt) {
+      const target = evt.target;
+      // Remove focus when an <option>-element was *clicked*, to improve the UX
+      // for mouse users (fixes bug 1300525 and issue 4923).
+      if (
+        this.value === self.pageScaleValue &&
+        target.tagName.toUpperCase() === "OPTION"
+      ) {
+        this.blur();
+      }
+    });
     // Suppress context menus for some controls.
     scaleSelect.oncontextmenu = noContextMenuHandler;