Fixes Accessibility issue: improve pop-up control focusing (#1133)

- Fixes #1071.
UniversalViewer · Oct 17, 2024 · 88178e3 · 88178e3
1 parent 9bb923a
commit 88178e3
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/content-handlers/iiif/modules/uv-shared-module/Dialogue.ts b/src/content-handlers/iiif/modules/uv-shared-module/Dialogue.ts
@@ -156,11 +156,11 @@ export class Dialogue<
       if ($defaultButton.length) {
         $defaultButton.focus();
       } else {
-        // if there's no default button, focus on the first visible input
-        const $input: JQuery = this.$element.find("input:visible").first();
+        // if there's no default button, focus on the first visible input or select element
+        const $firstVisibleElement: JQuery = this.$element.find("input:visible, select:visible").first();
 
-        if ($input.length) {
-          $input.focus();
+        if ($firstVisibleElement.length) {
+          $firstVisibleElement.focus();
         } else {
           // if there's no visible first input, focus on the close button
           this.$closeButton.focus();