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

Author: ok7sai

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

@@ -16,23 +16,20 @@ import {
 } from './event-manager';
 
 /**
- * Gets whether an event could be a faked click event dispatched by a screen reader
- * or generated by the browser when Enter or Space keys are pressed.
+ * Gets whether an event could be a simulated click event.
+ *
+ * Screen readers and keyboard activation (Enter/Space) often dispatch fake click
+ * events. We distinguish them by checking if `event.detail` is zero or if
+ * `event.pointerType` is missing.
  */
 export function isFakeClick(event: PointerEvent): boolean {
-  // Some screen readers will dispatch a fake click event when pressing enter or space on
-  // a clickable element. We can distinguish these events when `event.buttons` is zero, or
-  // `event.detail` is zero depending on the browser:
-  // - `event.buttons` works on Firefox, but fails on Chrome.
-  // - `detail` works on Chrome, but fails on Firefox.
-  // Also, fake events might not have a valid pointerType.
-  return event.buttons === 0 || event.detail === 0 || !event.pointerType;
+  return event.detail === 0 || !event.pointerType;
 }
 
 /**
  * An event manager that is specialized for handling click events.
  *
- * This manager should ONLY be used to handle pointer events. It explicitly
+ * This manager should ONLY be used to handle click events. It explicitly
  * filters out simulated click events generated by browsers when Enter or Space
  * keys are pressed, to avoid concurrent logic or overwriting selection state
  * when handling keyboard activation explicitly via KeyboardEventManager.