fix(aria/many): don't repeat keys

Author: adolgachev

src/aria/private/behaviors/event-manager/event-manager.ts

@@ -24,6 +24,7 @@ export interface EventWithModifiers extends Event {
  * This library has not yet had a need for stopPropagationImmediate.
  */
 export interface EventHandlerOptions {
+  handleRepeat?: boolean;
   stopPropagation: boolean;
   preventDefault: boolean;
 }

src/aria/private/behaviors/event-manager/keyboard-event-manager.ts

@@ -30,6 +30,7 @@ type KeyCode = string | SignalLike<string> | RegExp;
  */
 export class KeyboardEventManager<T extends KeyboardEvent> extends EventManager<T> {
   options: EventHandlerOptions = {
+    handleRepeat: false,
     preventDefault: true,
     stopPropagation: true,
   };
@@ -50,7 +51,7 @@ export class KeyboardEventManager<T extends KeyboardEvent> extends EventManager<
 
     this.configs.push({
       handler: handler,
-      matcher: event => this._isMatch(event, key, modifiers),
+      matcher: event => this._isMatch(event, key, modifiers, options),
       ...this.options,
       ...options,
     });
@@ -73,11 +74,20 @@ export class KeyboardEventManager<T extends KeyboardEvent> extends EventManager<
     };
   }
 
-  private _isMatch(event: T, key: KeyCode, modifiers: ModifierInputs) {
+  private _isMatch(
+    event: T,
+    key: KeyCode,
+    modifiers: ModifierInputs,
+    options?: Partial<EventHandlerOptions>,
+  ): boolean {
     if (!hasModifiers(event, modifiers)) {
       return false;
     }
 
+    if (event.repeat && !options?.handleRepeat) {
+      return false;
+    }
+
     if (key instanceof RegExp) {
       return key.test(event.key);
     }

src/aria/private/toolbar/toolbar.ts

@@ -80,10 +80,10 @@ export class ToolbarPattern<V> {
     const manager = new KeyboardEventManager();
 
     return manager
-      .on(this._nextKey, () => this.listBehavior.next())
-      .on(this._prevKey, () => this.listBehavior.prev())
-      .on(this._altNextKey, () => this._groupNext())
-      .on(this._altPrevKey, () => this._groupPrev())
+      .on(this._nextKey, () => this.listBehavior.next(), {handleRepeat: true})
+      .on(this._prevKey, () => this.listBehavior.prev(), {handleRepeat: true})
+      .on(this._altNextKey, () => this._groupNext(), {handleRepeat: true})
+      .on(this._altPrevKey, () => this._groupPrev(), {handleRepeat: true})
       .on(' ', () => this.select())
       .on('Enter', () => this.select())
       .on('Home', () => this.listBehavior.first())