fix(multiple): change delays to use ms (#32321)

* fix(multiple): change delays to use ms

* fixup! fix(multiple): change delays to use ms

Author: wagnermaciel

src/aria/listbox/listbox.spec.ts

@@ -724,7 +724,7 @@ describe('Listbox', () => {
       });
 
       it('should reset search term after typeaheadDelay', async () => {
-        setupListbox({options: getOptions(), focusMode, typeaheadDelay: 0.1});
+        setupListbox({options: getOptions(), focusMode, typeaheadDelay: 100});
 
         type('A');
         expect(isFocused(1)).toBe(true);
@@ -817,7 +817,7 @@ class ListboxExample {
   multi = false;
   wrap = true;
   selectionMode: 'follow' | 'explicit' = 'explicit';
-  typeaheadDelay = 0.5;
+  typeaheadDelay = 500;
 }
 
 @Component({

src/aria/listbox/listbox.ts

@@ -123,7 +123,7 @@ export class Listbox<V> {
   selectionMode = input<'follow' | 'explicit'>('follow');
 
   /** The amount of time before the typeahead search is reset. */
-  typeaheadDelay = input<number>(0.5); // Picked arbitrarily.
+  typeaheadDelay = input<number>(500); // Picked arbitrarily.
 
   /** Whether the listbox is disabled. */
   disabled = input(false, {transform: booleanAttribute});

src/aria/menu/menu.ts

@@ -192,8 +192,8 @@ export class Menu<V> {
   /** Whether the menu should wrap its items. */
   readonly wrap = input(true, {transform: booleanAttribute});
 
-  /** The delay in seconds before the typeahead buffer is cleared. */
-  readonly typeaheadDelay = input<number>(0.5); // Picked arbitrarily.
+  /** The delay in milliseconds before the typeahead buffer is cleared. */
+  readonly typeaheadDelay = input<number>(500); // Picked arbitrarily.
 
   /** Whether the menu is disabled. */
   readonly disabled = input(false, {transform: booleanAttribute});
@@ -222,8 +222,8 @@ export class Menu<V> {
   /** A callback function triggered when a menu item is selected. */
   onSelect = output<V>();
 
-  /** The delay in seconds before expanding sub-menus on hover. */
-  readonly expansionDelay = input<number>(0.1); // Arbitrarily chosen.
+  /** The delay in milliseconds before expanding sub-menus on hover. */
+  readonly expansionDelay = input<number>(100); // Arbitrarily chosen.
 
   constructor() {
     this._pattern = new MenuPattern({
@@ -344,8 +344,8 @@ export class MenuBar<V> {
   /** Whether the menu should wrap its items. */
   readonly wrap = input(true, {transform: booleanAttribute});
 
-  /** The delay in seconds before the typeahead buffer is cleared. */
-  readonly typeaheadDelay = input<number>(0.5);
+  /** The delay in milliseconds before the typeahead buffer is cleared. */
+  readonly typeaheadDelay = input<number>(500);
 
   /** The menu ui pattern instance. */
   readonly _pattern: MenuBarPattern<V>;

src/aria/private/behaviors/list-typeahead/list-typeahead.spec.ts

@@ -26,7 +26,7 @@ function getTypeahead(inputs: TestInputs = {}): ListTypeahead<TestItem> {
     focusManager,
     ...focusManager.inputs,
     items,
-    typeaheadDelay: signal(0.5),
+    typeaheadDelay: signal(500),
     ...inputs,
   });
 }

src/aria/private/behaviors/list-typeahead/list-typeahead.ts

@@ -74,7 +74,7 @@ export class ListTypeahead<T extends ListTypeaheadItem> {
     this.timeout = setTimeout(() => {
       this._query.set('');
       this._startIndex.set(undefined);
-    }, this.inputs.typeaheadDelay() * 1000);
+    }, this.inputs.typeaheadDelay());
 
     return true;
   }

src/aria/private/behaviors/list/list.spec.ts

@@ -24,7 +24,7 @@ describe('List Behavior', () => {
     return new List({
       values: inputs.values ?? signal([]),
       activeItem: signal(undefined),
-      typeaheadDelay: inputs.typeaheadDelay ?? signal(0.5),
+      typeaheadDelay: inputs.typeaheadDelay ?? signal(500),
       wrap: inputs.wrap ?? signal(true),
       disabled: inputs.disabled ?? signal(false),
       multi: inputs.multi ?? signal(false),
@@ -390,7 +390,7 @@ describe('List Behavior', () => {
     });
 
     it('should respect typeaheadDelay', async () => {
-      const {list} = getDefaultPatterns({typeaheadDelay: signal(0.1)});
+      const {list} = getDefaultPatterns({typeaheadDelay: signal(100)});
       list.search('b');
       expect(list.inputs.activeItem()).toBe(list.inputs.items()[2]); // Banana
       await delay(50); // Less than delay

src/aria/private/combobox/combobox.spec.ts

@@ -125,7 +125,7 @@ function getListboxPattern(
     values: signal(initialValue ? [initialValue] : []),
     combobox: signal(combobox) as any,
     activeItem: signal(undefined),
-    typeaheadDelay: signal(0.5),
+    typeaheadDelay: signal(500),
     wrap: signal(true),
     readonly: signal(false),
     disabled: signal(false),
@@ -169,7 +169,7 @@ function getTreePattern(
     values: signal(initialValue ? [initialValue] : []),
     combobox: signal(combobox) as any,
     activeItem: signal(undefined),
-    typeaheadDelay: signal(0.5),
+    typeaheadDelay: signal(500),
     wrap: signal(true),
     disabled: signal(false),
     softDisabled: signal(true),

src/aria/private/listbox/listbox.spec.ts

@@ -36,7 +36,7 @@ describe('Listbox Pattern', () => {
       items: inputs.items,
       values: inputs.values ?? signal([]),
       activeItem: signal(undefined),
-      typeaheadDelay: inputs.typeaheadDelay ?? signal(0.5),
+      typeaheadDelay: inputs.typeaheadDelay ?? signal(500),
       wrap: inputs.wrap ?? signal(true),
       readonly: inputs.readonly ?? signal(false),
       disabled: inputs.disabled ?? signal(false),

src/aria/private/menu/menu.spec.ts

@@ -61,7 +61,7 @@ function getMenuBarPattern(values: string[], opts?: {textDirection: 'ltr' | 'rtl
     selectionMode: signal('explicit'),
     values: signal([]),
     wrap: signal(true),
-    typeaheadDelay: signal(0.5),
+    typeaheadDelay: signal(500),
     softDisabled: signal(true),
     focusMode: signal('activedescendant'),
     element: signal(document.createElement('div')),
@@ -99,7 +99,7 @@ function getMenuPattern(
     items: items,
     parent: signal(parent) as any,
     activeItem: signal(undefined),
-    typeaheadDelay: signal(0.5),
+    typeaheadDelay: signal(500),
     wrap: signal(true),
     softDisabled: signal(true),
     multi: signal(false),

src/aria/private/menu/menu.ts

@@ -232,7 +232,7 @@ export class MenuPattern<V> {
       this._closeTimeout = setTimeout(() => {
         item.close();
         this._closeTimeout = undefined;
-      }, this.inputs.expansionDelay() * 1000);
+      }, this.inputs.expansionDelay());
     }
   }
 
@@ -243,7 +243,7 @@ export class MenuPattern<V> {
     this._openTimeout = setTimeout(() => {
       item.open();
       this._openTimeout = undefined;
-    }, this.inputs.expansionDelay() * 1000);
+    }, this.inputs.expansionDelay());
   }
 
   /** Handles mouseout events for the menu. */