@celestia-island/hikari 0.55.43 → 0.55.44

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@celestia-island/hikari",
-  "version": "0.55.43",
+  "version": "0.55.44",
   "private": false,
   "type": "module",
   "description": "Hikari Vue 3 component library — production-grade UI components based on shittim-chest design system",

package/src/composables/useContextMenu.test.tsx

@@ -385,6 +385,22 @@ describe("useContextMenu + HkContextMenuProvider", () => {
     unbind();
   });
 
+  it("keeps real right-clicks working inside the suppression window (only long-press synthesis is swallowed)", () => {
+    const target = document.createElement("div");
+    target.setAttribute("data-bind-target", "");
+    document.body.appendChild(target);
+    const { opens, api } = mockApi();
+    const unbind = bindContextMenu(target, api, (point) => {
+      return { x: point.x, y: point.y, items: [{ key: "a", label: "A" }] };
+    });
+    // Two genuine pointer right-clicks, ~0ms apart: BOTH open (a
+    // desktop user re-aiming fast must not lose the second menu).
+    target.dispatchEvent(new MouseEvent("contextmenu", { bubbles: true, cancelable: true, clientX: 10, clientY: 10 }));
+    target.dispatchEvent(new MouseEvent("contextmenu", { bubbles: true, cancelable: true, clientX: 30, clientY: 30 }));
+    expect(opens).toHaveLength(2);
+    unbind();
+  });
+
   it("lets a null build defer to outer bindings without preventing", () => {
     const target = document.createElement("div");
     target.setAttribute("data-bind-target", "");

package/src/composables/useContextMenu.ts

@@ -110,6 +110,12 @@ export function bindContextMenu(
   let holdTimer: ReturnType<typeof setTimeout> | null = null;
   let holdOrigin: { x: number; y: number } | null = null;
   let lastOpenedAt = 0;
+  /** The kind that last opened a menu — the suppression window only
+   *  guards the LONG-PRESS path (platforms synthesize a native
+   *  contextmenu right after ours). A desktop user right-clicking twice
+   *  within the window is two REAL gestures: the second one must open
+   *  its own menu, never be swallowed. */
+  let lastOpenKind: ContextTriggerPoint["kind"] | null = null;
 
   const cancelHold = () => {
     if (holdTimer !== null) {
@@ -123,6 +129,7 @@ export function bindContextMenu(
     const request = build(point);
     if (!request) return false;
     lastOpenedAt = performance.now();
+    lastOpenKind = point.kind;
     api.open(request);
     return true;
   };
@@ -130,7 +137,10 @@ export function bindContextMenu(
   const onContextMenu = (event: MouseEvent) => {
     // Our own long-press just opened the menu; a synthesized native
     // event for the same gesture must not re-open (or replace) it.
-    if (performance.now() - lastOpenedAt < CONTEXT_SUPPRESS_MS) {
+    if (
+      lastOpenKind === "longpress"
+      && performance.now() - lastOpenedAt < CONTEXT_SUPPRESS_MS
+    ) {
       event.preventDefault();
       return;
     }