stimeo-ui 0.16.0 → 0.18.0

package/dist/controllers/menu_controller.js

@@ -1,14 +1,185 @@
 import { Controller } from '@hotwired/stimulus';
 
-// src/controllers/menu_controller.ts
+function isAriaDisabled(element) {
+  for (let current = element; current; current = current.parentElement) {
+    if (current.getAttribute("aria-disabled") === "true") return true;
+  }
+  return false;
+}
 
-// src/utils/arrow_step.ts
 function isReservedArrowChord(event, allow = []) {
   if (!event.key.startsWith("Arrow")) return false;
   return event.altKey && !allow.includes("alt") || event.ctrlKey && !allow.includes("ctrl") || event.metaKey && !allow.includes("meta") || event.shiftKey && !allow.includes("shift");
 }
+function hasModifierChord(event) {
+  return event.altKey || event.ctrlKey || event.metaKey || event.shiftKey;
+}
+
+function setAttributeIfChanged(element, name, value, beforeWrite) {
+  if (element.getAttribute(name) === value) return false;
+  if (value === null) element.removeAttribute(name);
+  else element.setAttribute(name, value);
+  return true;
+}
+
+function sharedRegistry(key, create) {
+  const symbol = Symbol.for(key);
+  const scope = globalThis;
+  const existing = scope[symbol];
+  if (existing !== void 0) return existing;
+  const registry2 = create();
+  Object.defineProperty(globalThis, symbol, { value: registry2 });
+  return registry2;
+}
+
+var registry = sharedRegistry(
+  "stimeo-ui.attribute-lease.registry.v1",
+  () => ({ stacks: /* @__PURE__ */ new WeakMap(), names: /* @__PURE__ */ new Map() })
+);
+function isAttributeValue(value) {
+  return value === null || typeof value === "string";
+}
+function readLeaseRecord(element, record) {
+  try {
+    const parsed = JSON.parse(element.getAttribute(record) ?? "null");
+    if (!Array.isArray(parsed)) return void 0;
+    const [original, written, ...rest] = parsed;
+    const linked = rest[0] !== false;
+    const beneath = linked ? rest : rest.slice(1);
+    if (!beneath.every((under) => typeof under === "string")) return void 0;
+    return isAttributeValue(original) && isAttributeValue(written) ? { original, written, linked, beneath } : void 0;
+  } catch {
+    return void 0;
+  }
+}
+function attributeStack(element, attribute) {
+  const byAttribute = registry.stacks.get(element) ?? /* @__PURE__ */ new Map();
+  registry.stacks.set(element, byAttribute);
+  const known = byAttribute.get(attribute);
+  if (known) return known;
+  const found = [];
+  for (const record of registry.names.get(attribute) ?? []) {
+    const read = readLeaseRecord(element, record);
+    if (read) found.push({ ...read, record });
+  }
+  const depth = ({ beneath }) => beneath.filter((name) => found.some((other) => other.record === name)).length;
+  const stack = found.sort((one, other) => depth(one) - depth(other)).map(({ record, written, original, linked }, index) => ({
+    lease: null,
+    record,
+    written,
+    original,
+    linked: linked && index > 0
+  }));
+  byAttribute.set(attribute, stack);
+  return stack;
+}
+function findHold(stack, lease, record) {
+  return stack.find((held) => held.lease === lease) ?? stack.find((held) => held.lease === null && held.record === record);
+}
+var AttributeLease = class {
+  #attribute;
+  #record;
+  #base;
+  #preserveAuthor;
+  #held = /* @__PURE__ */ new Set();
+  constructor(attribute, owner, options = {}) {
+    this.#attribute = attribute;
+    this.#record = `data-${owner}-${attribute}-lease`;
+    this.#base = options.base === true;
+    this.#preserveAuthor = options.preserveAuthor === true;
+    const names = registry.names;
+    names.set(attribute, (names.get(attribute) ?? /* @__PURE__ */ new Set()).add(this.#record));
+  }
+  write(element, value) {
+    const stack = attributeStack(element, this.#attribute);
+    const hold = this.#take(stack) ?? this.#acquire(element, stack);
+    this.#held.add(element);
+    const shows = !this.#covered(element, stack, hold);
+    hold.written = value;
+    this.#mark(element, stack);
+    if (shows) this.#reflect(element, value);
+  }
+  return(element) {
+    const held = this.#held.delete(element);
+    const known = registry.stacks.get(element)?.has(this.#attribute);
+    const stack = held || known || element.hasAttribute(this.#record) ? attributeStack(element, this.#attribute) : [];
+    const hold = this.#take(stack);
+    if (!hold) return false;
+    const carries = element.getAttribute(this.#attribute) === hold.written;
+    const owns = carries && !this.#covered(element, stack, hold);
+    const index = stack.indexOf(hold);
+    stack.splice(index, 1);
+    const above = stack[index];
+    if (above) above.linked &&= hold.linked;
+    this.#mark(element, stack);
+    if (!owns) return true;
+    const below = hold.linked ? stack[index - 1] : void 0;
+    this.#reflect(element, below ? below.written : hold.original);
+    return true;
+  }
+  returnAll() {
+    for (const element of Array.from(this.#held)) this.return(element);
+  }
+  #take(stack) {
+    const hold = findHold(stack, this, this.#record);
+    if (hold) hold.lease = this;
+    return hold;
+  }
+  #acquire(element, stack) {
+    const current = element.getAttribute(this.#attribute);
+    const top = stack.at(-1);
+    const bottom = stack[0];
+    const over = top?.written === current ? top : void 0;
+    const from = this.#base ? this.#preserveAuthor && !stack.some((held) => held.written === current) ? void 0 : bottom : over;
+    const hold = {
+      lease: this,
+      record: this.#record,
+      written: current,
+      original: from ? from.original : current,
+      linked: !this.#base && over !== void 0
+    };
+    if (this.#base && bottom) {
+      bottom.linked = true;
+      stack.unshift(hold);
+    } else stack.push(hold);
+    return hold;
+  }
+  #covered(element, stack, hold) {
+    const current = element.getAttribute(this.#attribute);
+    return stack.slice(stack.indexOf(hold) + 1).some((above) => above.written === current);
+  }
+  #mark(element, stack) {
+    const records = /* @__PURE__ */ new Map([[this.#record, null]]);
+    stack.forEach((hold, index) => {
+      const names = stack.slice(0, index).map((under) => under.record);
+      const beneath = [...new Set(names.reverse())].filter((name) => name !== hold.record);
+      const marker = hold.linked || beneath.length === 0 ? [] : [false];
+      records.set(
+        hold.record,
+        hold.written === hold.original && beneath.length === 0 && index === stack.length - 1 ? null : JSON.stringify([hold.original, hold.written, ...marker, ...beneath])
+      );
+    });
+    for (const [name, recorded] of records) {
+      setAttributeIfChanged(element, name, recorded);
+    }
+  }
+  #reflect(element, value) {
+    setAttributeIfChanged(element, this.#attribute, value);
+  }
+};
+
+var ClickVerdicts = class {
+  #verdicts = /* @__PURE__ */ new WeakMap();
+  record(event, entries, refused) {
+    this.#verdicts.set(event, { entries, refused });
+  }
+  for(event, entry) {
+    const verdict = event && this.#verdicts.get(event);
+    if (!verdict?.entries.includes(entry)) return void 0;
+    return verdict.refused;
+  }
+};
 
-// src/utils/escape_layer.ts
 function claimsWhileFocusWithin(element) {
   return () => {
     const active = element.ownerDocument.activeElement;
@@ -16,89 +187,68 @@ function claimsWhileFocusWithin(element) {
   };
 }
 var EscapeLayer = class _EscapeLayer {
-  static #registries = /* @__PURE__ */ new WeakMap();
-  #ownerDocument = null;
-  /** Dismissal callback while active; `null` when inactive. */
-  #onDismiss = null;
-  /** Live predicate deciding whether the layer claims a press; `null` = always. */
-  #claims = null;
-  /**
-   * Activates this layer at the top of its document's Escape stack, installing
-   * the document's shared resolver listener if this is its first layer.
-   * Re-activating an already-active layer moves it to the top.
-   */
+  static #registries = sharedRegistry(
+    "stimeo-ui.escape-layer.registry.v1",
+    () => /* @__PURE__ */ new WeakMap()
+  );
+  #active = null;
   activate(ownerDocument = document, options) {
     this.deactivate();
-    let registry = _EscapeLayer.#registries.get(ownerDocument);
-    if (!registry) {
-      registry = _EscapeLayer.#createRegistry();
-      _EscapeLayer.#registries.set(ownerDocument, registry);
-      ownerDocument.addEventListener("keydown", registry.onKeydown);
+    let registry2 = _EscapeLayer.#registries.get(ownerDocument);
+    if (!registry2) {
+      registry2 = _EscapeLayer.#createRegistry();
+      _EscapeLayer.#registries.set(ownerDocument, registry2);
+      ownerDocument.addEventListener("keydown", registry2.onKeydown);
     }
-    registry.stack.push(this);
-    this.#ownerDocument = ownerDocument;
-    this.#onDismiss = options.onDismiss;
-    this.#claims = options.claims ?? null;
-  }
-  /**
-   * Removes this layer from its document's Escape stack, uninstalling the
-   * shared listener when the stack empties. Safe to call when inactive.
-   */
+    const entry = { onDismiss: options.onDismiss, claims: options.claims ?? null };
+    registry2.stack.push(entry);
+    this.#active = { document: ownerDocument, entry };
+  }
   deactivate() {
-    const ownerDocument = this.#ownerDocument;
-    if (!ownerDocument) return;
-    const registry = _EscapeLayer.#registries.get(ownerDocument);
-    if (registry) {
-      const index = registry.stack.lastIndexOf(this);
-      if (index >= 0) registry.stack.splice(index, 1);
-      if (registry.stack.length === 0) {
-        ownerDocument.removeEventListener("keydown", registry.onKeydown);
-        _EscapeLayer.#registries.delete(ownerDocument);
+    const active = this.#active;
+    if (!active) return;
+    const registry2 = _EscapeLayer.#registries.get(active.document);
+    if (registry2) {
+      const index = registry2.stack.lastIndexOf(active.entry);
+      if (index >= 0) registry2.stack.splice(index, 1);
+      if (registry2.stack.length === 0) {
+        active.document.removeEventListener("keydown", registry2.onKeydown);
+        _EscapeLayer.#registries.delete(active.document);
       }
     }
-    this.#ownerDocument = null;
-    this.#onDismiss = null;
-    this.#claims = null;
-  }
-  /**
-   * Whether this active layer would own a press right now: it is the topmost
-   * layer whose {@link EscapeLayerOptions.claims} passes. Exposed for tests
-   * and diagnostics — production dismissal goes through the shared listener.
-   */
+    this.#active = null;
+  }
   get ownsEscape() {
-    const ownerDocument = this.#ownerDocument;
-    if (!ownerDocument) return false;
-    const registry = _EscapeLayer.#registries.get(ownerDocument);
-    if (!registry) return false;
-    return _EscapeLayer.#resolveOwner(registry.stack) === this;
+    const active = this.#active;
+    if (!active) return false;
+    const registry2 = _EscapeLayer.#registries.get(active.document);
+    if (!registry2) return false;
+    return _EscapeLayer.#resolveOwner(registry2.stack) === active.entry;
   }
-  /** Builds a document's registry with its shared resolver listener. */
   static #createRegistry() {
-    const registry = {
+    const registry2 = {
       stack: [],
       onKeydown: (event) => {
         if (event.key !== "Escape" || event.defaultPrevented || event.isComposing) return;
-        const owner = _EscapeLayer.#resolveOwner(registry.stack);
+        const owner = _EscapeLayer.#resolveOwner(registry2.stack);
         if (!owner) return;
         event.preventDefault();
-        owner.#onDismiss?.();
+        owner.onDismiss();
       }
     };
-    return registry;
+    return registry2;
   }
-  /** The topmost stack layer whose claims predicate passes, or `null`. */
   static #resolveOwner(stack) {
     for (let index = stack.length - 1; index >= 0; index--) {
       const layer = stack[index];
       if (!layer) continue;
-      if (layer.#claims && !layer.#claims()) continue;
+      if (layer.claims && !layer.claims()) continue;
       return layer;
     }
     return null;
   }
 };
 
-// src/utils/event_owner.ts
 function ownerIndex(candidates, node) {
   if (!(node instanceof Node)) return -1;
   return candidates.findIndex((candidate) => candidate.contains(node));
@@ -106,44 +256,40 @@ function ownerIndex(candidates, node) {
 function ownerOf(candidates, node) {
   return candidates[ownerIndex(candidates, node)] ?? null;
 }
+function ownersOf(candidates, node) {
+  if (!(node instanceof Node)) return [];
+  return candidates.filter((candidate) => candidate.contains(node));
+}
+
+var ListenerSet = class {
+  #abort = new AbortController();
+  add(target, type, handler, options) {
+    target.addEventListener(type, handler, { ...options, signal: this.#abort.signal });
+  }
+  dispose() {
+    this.#abort.abort();
+    this.#abort = new AbortController();
+  }
+};
 
-// src/utils/safe_timeout.ts
 var TimerRegistry = class {
-  /** Live timer ids that have not yet been cleared (or, for timeouts, fired). */
   ids = /* @__PURE__ */ new Set();
-  /**
-   * Cancels a single tracked timer.
-   *
-   * No-ops if the id is unknown (already cleared, fired, or never owned by this
-   * registry), so callers can clear defensively without guarding.
-   */
   clear(id) {
     if (this.ids.delete(id)) {
       this.cancel(id);
     }
   }
-  /**
-   * Cancels every tracked timer. Call this from a controller's `disconnect()`
-   * to guarantee no timer outlives the element.
-   */
   clearAll() {
     for (const id of this.ids) {
       this.cancel(id);
     }
     this.ids.clear();
   }
-  /** Number of timers currently tracked (pending). */
   get size() {
     return this.ids.size;
   }
 };
 var SafeTimeout = class extends TimerRegistry {
-  /**
-   * Schedules `callback` after `delay` ms and returns the timer id.
-   *
-   * The id is removed from the registry automatically when the timeout fires,
-   * so {@link TimerRegistry.size | size} reflects only still-pending timers.
-   */
   set(callback, delay) {
     const id = this.schedule(() => {
       this.ids.delete(id);
@@ -160,7 +306,6 @@ var SafeTimeout = class extends TimerRegistry {
   }
 };
 
-// src/utils/state_reason.ts
 var FOCUS_EVENTS = /* @__PURE__ */ new Set(["blur", "focus", "focusin", "focusout"]);
 var POINTER_EVENTS = /* @__PURE__ */ new Set(["mouseenter", "mouseleave", "pointerenter", "pointerleave"]);
 function stateReasonFor(event) {
@@ -170,7 +315,6 @@ function stateReasonFor(event) {
   return "user";
 }
 
-// src/controllers/menu_controller.ts
 var MenuController = class extends Controller {
   static targets = ["trigger", "menu", "item"];
   static actions = [
@@ -183,38 +327,46 @@ var MenuController = class extends Controller {
   ];
   static events = ["close", "open"];
   #timers = new SafeTimeout();
-  /** Escape-stack membership while open; the shared resolver dismisses via it. */
   #escapeLayer = new EscapeLayer();
-  /**
-   * The item a click started on, recorded in the capture pass. Read once by the
-   * delegated bubble listener, which must not re-derive it: consumer handlers run
-   * in between and may detach the item.
-   */
+  #menuHidden = new AttributeLease("hidden", this.identifier);
+  #expanded = new AttributeLease("aria-expanded", this.identifier);
+  #menu = null;
   #clickOwner = null;
-  /** Clicks already turned into an activation, so the two paths run it once. */
+  #clickVerdicts = new ClickVerdicts();
   #activated = /* @__PURE__ */ new WeakSet();
-  /** Whether state moves are reported: set once `connect()` settled the baseline. */
+  #connected = false;
   #reporting = false;
-  /** Starts closed and registers activation / outside-click listeners. */
+  #listeners = new ListenerSet();
   connect() {
     this.#close("api");
-    this.element.addEventListener("click", this.#onItemClickCapture, true);
-    this.element.addEventListener("click", this.#onDelegatedItemClick);
-    this.element.addEventListener("keydown", this.#onDelegatedItemKeydown);
-    document.addEventListener("click", this.#onOutsideClick, true);
+    this.#listeners.add(this.element, "click", this.#onItemClickCapture, { capture: true });
+    this.#listeners.add(this.element, "click", this.#onDelegatedItemClick);
+    this.#listeners.add(this.element, "keydown", this.#onDelegatedItemKeydown);
+    this.#listeners.add(document, "click", this.#onOutsideClick, { capture: true });
+    this.#connected = true;
     this.#reporting = true;
   }
-  /** Releases the listeners, stack membership, and any pending Tab-close task. */
   disconnect() {
+    this.#connected = false;
     this.#reporting = false;
     this.#timers.clearAll();
     this.#escapeLayer.deactivate();
-    this.element.removeEventListener("click", this.#onItemClickCapture, true);
-    this.element.removeEventListener("click", this.#onDelegatedItemClick);
-    this.element.removeEventListener("keydown", this.#onDelegatedItemKeydown);
-    document.removeEventListener("click", this.#onOutsideClick, true);
+    this.#listeners.dispose();
+  }
+  triggerTargetConnected() {
+    if (this.#connected) this.#reflectTrigger();
+  }
+  triggerTargetDisconnected(trigger) {
+    if (!this.triggerTargets.includes(trigger)) this.#expanded.return(trigger);
+    if (this.#connected) this.#reflectTrigger();
+  }
+  menuTargetConnected() {
+    this.#adoptMenu();
+  }
+  menuTargetDisconnected(menu) {
+    this.#adoptMenu();
+    if (!this.menuTargets.includes(menu)) this.#menuHidden.return(menu);
   }
-  /** Toggles the menu open/closed. Bound via `data-action` (click). */
   toggle(event) {
     if (this.#isOpen) {
       this.close(event);
@@ -223,45 +375,52 @@ var MenuController = class extends Controller {
       this.#focusFirst();
     }
   }
-  /** Opens the menu and reflects the expanded state on the trigger. */
   open(event) {
     this.#open(stateReasonFor(event));
   }
-  /** Closes the menu and reflects the collapsed state on the trigger. */
   close(event) {
     this.#close(stateReasonFor(event));
   }
-  /** Opens the menu, reflects the expanded state, and reports a move. */
   #open(reason) {
     this.#timers.clearAll();
     if (!this.hasMenuTarget) return;
+    const menu = this.menuTarget;
     const was = this.#isOpen;
     this.#escapeLayer.activate(document, {
       onDismiss: () => this.#closeAndRestore("escape"),
       claims: claimsWhileFocusWithin(this.element)
     });
-    this.menuTarget.hidden = false;
-    if (this.hasTriggerTarget) this.triggerTarget.setAttribute("aria-expanded", "true");
+    this.#menuHidden.write(menu, null);
+    this.#menu = menu;
+    if (this.hasTriggerTarget) this.#expanded.write(this.triggerTarget, "true");
     if (!was && this.#reporting) this.dispatch("open", { detail: { reason }, cancelable: false });
   }
-  /** Closes the menu, reflects the collapsed state, and reports a move. */
   #close(reason) {
     this.#timers.clearAll();
     this.#escapeLayer.deactivate();
-    if (!this.hasMenuTarget) return;
+    const menu = this.hasMenuTarget ? this.menuTarget : null;
+    this.#menu = menu;
+    if (!menu) return;
     const was = this.#isOpen;
-    this.menuTarget.hidden = true;
-    if (this.hasTriggerTarget) this.triggerTarget.setAttribute("aria-expanded", "false");
+    this.#menuHidden.write(menu, "");
+    if (this.hasTriggerTarget) this.#expanded.write(this.triggerTarget, "false");
     if (was && this.#reporting) this.dispatch("close", { detail: { reason }, cancelable: false });
   }
-  /**
-   * Opens the menu with the keyboard per the APG (Down → first, Up → last).
-   *
-   * Enter/Space are intentionally not handled here: on a native `<button>`
-   * trigger the browser turns them into a click, which already runs
-   * {@link toggle} (open + focus first item). Handling them again here would
-   * open and then immediately re-toggle the menu.
-   */
+  #adoptMenu() {
+    if (!this.#connected) return;
+    const menu = this.hasMenuTarget ? this.menuTarget : null;
+    const previous = this.#menu;
+    this.#menu = menu;
+    if (!menu) {
+      this.#escapeLayer.deactivate();
+    } else if (menu !== previous) {
+      this.#menuHidden.write(menu, previous?.hidden ?? true ? "" : null);
+    }
+    this.#reflectTrigger();
+  }
+  #reflectTrigger() {
+    if (this.hasTriggerTarget) this.#expanded.write(this.triggerTarget, String(this.#isOpen));
+  }
   onTriggerKeydown(event) {
     if (event.defaultPrevented) return;
     if (isReservedArrowChord(event)) return;
@@ -275,18 +434,13 @@ var MenuController = class extends Controller {
       this.#focusLast();
     }
   }
-  /** Implements roving focus and closing keys inside the menu. */
   onItemKeydown(event) {
     this.#handleItemKeydown(event, event.currentTarget);
   }
-  /**
-   * The roving-focus body, shared by the per-element action and its delegated
-   * twin. `from` is the item the key belongs to: the bound element for the
-   * action, the resolved owner of the event target for the delegate.
-   */
   #handleItemKeydown(event, from) {
     if (event.defaultPrevented) return;
     if (isReservedArrowChord(event)) return;
+    if ((event.key === "Home" || event.key === "End") && hasModifierChord(event)) return;
     const items = this.#navigableItems;
     const currentIndex = items.indexOf(from);
     switch (event.key) {
@@ -312,120 +466,80 @@ var MenuController = class extends Controller {
         break;
     }
   }
-  /**
-   * Closes the menu after an item is activated. Bound via `data-action`, and
-   * also reached from the delegated listener.
-   *
-   * Markup that carries the per-element action *and* gets the delegate would run
-   * this twice for one gesture. `close()` writes the state hooks (`hidden`,
-   * `aria-expanded`), and an identical reassign still queues a MutationRecord,
-   * so a second pass is observable to anyone watching them. The event is
-   * therefore claimed: the path that gets there first does the work, the other
-   * one finds it claimed and returns. A programmatic call with no event always
-   * runs.
-   */
   activate(event) {
-    if (event !== void 0) {
-      if (this.#activated.has(event)) return;
-      this.#activated.add(event);
-    }
+    if (event === void 0) this.#closeAndRestore("select");
+    else this.#activateItem(this.#itemFrom(event.target), event);
+  }
+  #activateItem(item, event) {
+    if (item !== null && this.#refuses(item, event)) return;
+    if (this.#activated.has(event)) return;
+    this.#activated.add(event);
     this.#closeAndRestore("select");
   }
-  /** Closes and returns focus to the trigger (Escape / item-activation path). */
+  #refuses(item, event) {
+    const recorded = this.#clickVerdicts.for(event, item);
+    if (recorded !== void 0) return recorded;
+    return ownersOf(this.itemTargets, event.target).some(
+      (owner) => this.#isActivationDisabled(owner)
+    );
+  }
   #closeAndRestore(reason) {
     this.#close(reason);
     if (this.hasTriggerTarget) this.triggerTarget.focus();
   }
-  /** Closes the menu when a click lands outside the controller's element. */
   #onOutsideClick = (event) => {
     if (this.#isOpen && !this.element.contains(event.target)) this.#close("outside");
   };
-  /**
-   * Delegated twin of {@link onItemKeydown}, bound on the controller element.
-   *
-   * An item that arrives *after* connect — `stimeo--overflow-menu` moves toolbar controls
-   * into this menu at runtime — carries whatever `data-action` its author wrote,
-   * and that is exactly the binding a consumer forgets, because the markup that
-   * declares the item lives nowhere near the menu. Listening on the container
-   * makes membership in the `item` target enough to be operable. The per-element
-   * action stays supported and is not double-handled: see
-   * {@link #handleItemKeydown}.
-   */
   #onDelegatedItemKeydown = (event) => {
     const item = this.#itemFrom(event.target);
     if (item === null) return;
     this.#handleItemKeydown(event, item);
   };
-  /**
-   * Delegated twin of {@link activate}, for the same reason as the keydown one.
-   *
-   * It works from the item the capture pass recorded, not from a fresh lookup:
-   * by the time a click bubbles up here, a consumer handler may already have
-   * detached the row it lives in, and `itemTargets` is a live query that would
-   * then resolve nothing — the activation would be lost and the menu left open
-   * over an item that no longer exists. `aria-disabled` items never reach this
-   * listener at all: the capture pass stops the click outright.
-   */
   #onDelegatedItemClick = (event) => {
     const item = this.#clickOwner;
     this.#clickOwner = null;
     if (item === null) return;
-    this.activate(event);
+    this.#activateItem(item, event);
   };
-  /** The item target that is, or contains, `node`; `null` when it is neither. */
   #itemFrom(node) {
     return ownerOf(this.itemTargets, node);
   }
-  /**
-   * First look at every click inside the controller, and the only point at which
-   * the DOM is still the one the user clicked. Two jobs: record which item owns
-   * the click for the delegate above, and stop an `aria-disabled` command before
-   * it reaches consumer handlers (native Enter/Space activation synthesizes a
-   * click and is blocked here too).
-   */
   #onItemClickCapture = (event) => {
-    const target = event.target;
-    if (!(target instanceof Node)) {
-      this.#clickOwner = null;
-      return;
-    }
-    const item = this.#itemFrom(target);
-    this.#clickOwner = item;
-    if (item === null || item.getAttribute("aria-disabled") !== "true") return;
+    const items = ownersOf(this.itemTargets, event.target);
+    this.#clickOwner = items[0] ?? null;
+    this.#clickVerdicts.record(
+      event,
+      items,
+      items.some((item) => this.#isActivationDisabled(item))
+    );
+    if (!items.some((item) => isAriaDisabled(item))) return;
     this.#clickOwner = null;
     event.preventDefault();
     event.stopImmediatePropagation();
   };
-  /** Moves focus to the first navigable item (no-op if none). */
   #focusFirst() {
     if (!this.#isOpen) return;
     this.#navigableItems[0]?.focus();
   }
-  /** Moves focus to the last navigable item (no-op if none). */
   #focusLast() {
     if (!this.#isOpen) return;
     const items = this.#navigableItems;
     items[items.length - 1]?.focus();
   }
-  /** Menu items eligible for roving focus (excludes disabled / hidden). */
   get #navigableItems() {
     return this.itemTargets.filter((item) => this.#isNavigable(item));
   }
-  /**
-   * An item can take roving focus unless it is `hidden` or a natively `disabled`
-   * form control. `aria-disabled` remains focusable for discoverability; capture
-   * suppresses activation. CSS-only visibility is the consumer's responsibility.
-   */
   #isNavigable(item) {
     if (item.hasAttribute("hidden")) return false;
     return !item.disabled;
   }
-  /** Whether the menu is currently visible. */
+  #isActivationDisabled(item) {
+    return !this.#isNavigable(item) || isAriaDisabled(item);
+  }
   get #isOpen() {
     return this.hasMenuTarget && !this.menuTarget.hidden;
   }
 };
 
 export { MenuController };
-//# sourceMappingURL=menu_controller.js.map
 //# sourceMappingURL=menu_controller.js.map