Bail on escape key press if focus is on anything (other than a link)

Fixes #12811.

CHANGES.rst

@@ -87,6 +87,7 @@ Bugs fixed
   file URL (user-defined base URL of an intersphinx project are left untouched
   even if they end with double forward slashes).
   Patch by Bénédikt Tran.
+* #12811: Bail on escape key press if focus is on anything (other than a link)
 
 Testing
 -------

sphinx/themes/basic/static/doctools.js

@@ -10,13 +10,6 @@
  */
 "use strict";
 
-const BLACKLISTED_KEY_CONTROL_ELEMENTS = new Set([
-  "TEXTAREA",
-  "INPUT",
-  "SELECT",
-  "BUTTON",
-]);
-
 const _ready = (callback) => {
   if (document.readyState !== "loading") {
     callback();
@@ -111,8 +104,9 @@ const Documentation = {
       return;
 
     document.addEventListener("keydown", (event) => {
-      // bail for input elements
-      if (BLACKLISTED_KEY_CONTROL_ELEMENTS.has(document.activeElement.tagName)) return;
+      // bail if browser focus is on anything but a link
+      const focusEl = document.activeElement;
+      if (focusEl && (focusEl !== document.body || focusEl.tagName !== "A")) return;
       // bail with special keys
       if (event.altKey || event.ctrlKey || event.metaKey) return;
 

sphinx/themes/basic/static/sphinx_highlight.js

@@ -133,8 +133,9 @@ const SphinxHighlight = {
     if (!DOCUMENTATION_OPTIONS.ENABLE_SEARCH_SHORTCUTS) return;
 
     document.addEventListener("keydown", (event) => {
-      // bail for input elements
-      if (BLACKLISTED_KEY_CONTROL_ELEMENTS.has(document.activeElement.tagName)) return;
+      // bail if browser focus is on anything but a link
+      const focusEl = document.activeElement;
+      if (focusEl && (focusEl !== document.body || focusEl.tagName !== "A")) return;
       // bail with special keys
       if (event.shiftKey || event.altKey || event.ctrlKey || event.metaKey) return;
       if (DOCUMENTATION_OPTIONS.ENABLE_SEARCH_SHORTCUTS && (event.key === "Escape")) {