fix(reactive-controllers): complex element delegates focus to active …

…child WAI ARIA APG
adobe · Jan 23, 2025 · 350e9ba · 350e9ba
1 parent 3876c36
commit 350e9ba
Showing 1 changed file with 8 additions and 12 deletions.
diff --git a/tools/reactive-controllers/src/RovingTabindex.ts b/tools/reactive-controllers/src/RovingTabindex.ts
@@ -45,18 +45,14 @@ export class RovingTabindexController<
     }
 
     manageTabindexes(): void {
-        if (this.focused) {
-            this.updateTabindexes(() => ({ tabIndex: -1 }));
-        } else {
-            this.updateTabindexes((el: HTMLElement): UpdateTabIndexes => {
-                return {
-                    removeTabIndex:
-                        el.contains(this.focusInElement) &&
-                        el !== this.focusInElement,
-                    tabIndex: el === this.focusInElement ? 0 : -1,
-                };
-            });
-        }
+        this.updateTabindexes((el: HTMLElement): UpdateTabIndexes => {
+            return {
+                removeTabIndex:
+                    el.contains(this.focusInElement) &&
+                    el !== this.focusInElement,
+                tabIndex: el === this.focusInElement ? 0 : -1,
+            };
+        });
     }
 
     updateTabindexes(getTabIndex: (el: HTMLElement) => UpdateTabIndexes): void {