Fix active item on focus

and refactor tests for listbox
hokulea · Aug 5, 2024 · 9026e99 · 9026e99
1 parent c4362f5
commit 9026e99
Show file tree

Hide file tree

Showing 28 changed files with 1,026 additions and 884 deletions.
diff --git a/packages/aria-voyager/src/navigation-patterns/active-descendent-strategy.ts b/packages/aria-voyager/src/navigation-patterns/active-descendent-strategy.ts
@@ -33,11 +33,11 @@ export class ActiveDescendentStrategy implements NavigationPattern, FocusStrateg
   }
 
   handleFocus() {
-    const multiSelection =
-      this.control.capabilities.multiSelection && this.control.options.multiple;
+    // const multiSelection =
+    //   this.control.capabilities.multiSelection && this.control.options.multiple;
     const selectionPresent = this.control.selection.length > 0;
 
-    if (multiSelection && selectionPresent) {
+    if (selectionPresent) {
       this.activateItem(this.control.selection[0]);
     } else {
       this.activateItem(this.control.items[0]);

diff --git a/packages/aria-voyager/tests/listbox/multi-selection.test.ts b/packages/aria-voyager/tests/listbox/multi-selection.test.ts