Fixed content tabs stealing keyboard access to search

Author: squidfunk

…ssets/javascripts/bundle.bc35569b.min.js …ssets/javascripts/bundle.801e7afd.min.jsmaterial/assets/javascripts/bundle.bc35569b.min.js renamed to material/assets/javascripts/bundle.801e7afd.min.js material/assets/javascripts/bundle.bc35569b.min.js renamed to material/assets/javascripts/bundle.801e7afd.min.js

@@ -198,7 +198,7 @@ <h1>{{ page.title | d(config.site_name, true)}}</h1>
         "base": base_url,
         "features": features,
         "translations": {},
-        "search": "assets/javascripts/workers/search.e99e714c.min.js" | url,
+        "search": "assets/javascripts/workers/search.8397ff9e.min.js" | url,
         "version": config.extra.version or None
       } -%}
       {%- set translations = app.translations -%}
@@ -225,7 +225,7 @@ <h1>{{ page.title | d(config.site_name, true)}}</h1>
       </script>
     {% endblock %}
     {% block scripts %}
-      <script src="{{ 'assets/javascripts/bundle.bc35569b.min.js' | url }}"></script>
+      <script src="{{ 'assets/javascripts/bundle.801e7afd.min.js' | url }}"></script>
       {% for path in config["extra_javascript"] %}
         <script src="{{ path | url }}"></script>
       {% endfor %}

…s/javascripts/bundle.bc35569b.min.js.map …s/javascripts/bundle.801e7afd.min.js.mapmaterial/assets/javascripts/bundle.bc35569b.min.js.map renamed to material/assets/javascripts/bundle.801e7afd.min.js.map material/assets/javascripts/bundle.bc35569b.min.js.map renamed to material/assets/javascripts/bundle.801e7afd.min.js.map

@@ -16,5 +16,5 @@
 {% endblock %}
 {% block scripts %}
   {{ super() }}
-  <script src="{{ 'overrides/assets/javascripts/bundle.bb368c8a.min.js' | url }}"></script>
+  <script src="{{ 'overrides/assets/javascripts/bundle.f2c7354c.min.js' | url }}"></script>
 {% endblock %}

…vascripts/workers/search.e99e714c.min.js …vascripts/workers/search.8397ff9e.min.jsmaterial/assets/javascripts/workers/search.e99e714c.min.js renamed to material/assets/javascripts/workers/search.8397ff9e.min.js material/assets/javascripts/workers/search.e99e714c.min.js renamed to material/assets/javascripts/workers/search.8397ff9e.min.js

@@ -56,16 +56,26 @@ export interface Keyboard {
  * Check whether an element may receive keyboard input
  *
  * @param el - Element
+ * @param type - Key type
  *
  * @returns Test result
  */
-function isSusceptibleToKeyboard(el: HTMLElement): boolean {
-  switch (el.tagName) {
+function isSusceptibleToKeyboard(
+  el: HTMLElement, type: string
+): boolean {
+  switch (el.constructor) {
 
-    /* Form elements */
-    case "INPUT":
-    case "SELECT":
-    case "TEXTAREA":
+    /* Input elements */
+    case HTMLInputElement:
+      /* @ts-expect-error - omit unnecessary type cast */
+      if (el.type === "radio")
+        return /^Arrow/.test(type)
+      else
+        return true
+
+    /* Select element and textarea */
+    case HTMLSelectElement:
+    case HTMLTextAreaElement:
       return true
 
     /* Everything else */
@@ -95,11 +105,11 @@ export function watchKeyboard(): Observable<Keyboard> {
           ev.stopPropagation()
         }
       } as Keyboard)),
-      filter(({ mode }) => {
+      filter(({ mode, type }) => {
         if (mode === "global") {
           const active = getActiveElement()
           if (typeof active !== "undefined")
-            return !isSusceptibleToKeyboard(active)
+            return !isSusceptibleToKeyboard(active, type)
         }
         return true
       }),

…ripts/workers/search.e99e714c.min.js.map …ripts/workers/search.8397ff9e.min.js.mapmaterial/assets/javascripts/workers/search.e99e714c.min.js.map renamed to material/assets/javascripts/workers/search.8397ff9e.min.js.map material/assets/javascripts/workers/search.e99e714c.min.js.map renamed to material/assets/javascripts/workers/search.8397ff9e.min.js.map

@@ -163,7 +163,7 @@ export async function handler(
  * Worker
  * ------------------------------------------------------------------------- */
 
-/* @ts-ignore - expose Lunr.js in global scope, or stemmers will not work */
+/* @ts-expect-error - expose Lunr.js in global scope, or stemmers won't work */
 self.lunr = lunr
 
 /* Handle messages */