fixup! refactor(multiple): migrate Listbox and Tabs to ClickEventManager

Author: ok7sai

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

@@ -26,6 +26,14 @@ export function isFakeClick(event: PointerEvent): boolean {
   return event.detail === 0 || !event.pointerType;
 }
 
+/**
+ * Gets whether an event is a programmatic click (e.g. triggered by .click() or .dispatchEvent()).
+ * Programmatic events are untrusted.
+ */
+export function isProgrammaticClick(event: Event): boolean {
+  return !event.isTrusted;
+}
+
 /**
  * An event manager that is specialized for handling click events.
  *
@@ -78,6 +86,7 @@ export class ClickEventManager<T extends PointerEvent> extends EventManager<T> {
   }
 
   _isMatch(event: T, modifiers: ModifierInputs) {
-    return !isFakeClick(event) && hasModifiers(event, modifiers);
+    const isAllowed = isProgrammaticClick(event) || !isFakeClick(event);
+    return isAllowed && hasModifiers(event, modifiers);
   }
 }

src/aria/tabs/tabs.spec.ts

@@ -53,10 +53,6 @@ describe('Tabs', () => {
     target.dispatchEvent(
       new PointerEvent('click', {
         bubbles: true,
-        detail: 1,
-        pointerType: 'mouse',
-        clientX: 1,
-        clientY: 1,
         ...eventInit,
       }),
     );