@jsenv/navi 0.29.400 → 0.29.402

package/dist/dev/jsenv_navi.js

@@ -6288,6 +6288,9 @@ const CONTROL_ATTRIBUTE_SET = new Set([
   "onnavi_command",
   "onnavi_request_open",
   "onnavi_request_close",
+  // The press announcing that request (see announceOpeningPress in
+  // commands.js): heard where the request itself is.
+  "onnavi_open_press",
 
   "data-callout-arrow-x",
   "data-callout-point-to-border-box",
@@ -38554,8 +38557,8 @@ const armOutsidePressClose = (
  * contents; a press is the start of one opening.
  *
  * Only what opens the popup can say its press is one, and it says so as the
- * press starts, with a `navi_open_press` on the popup (see
- * announceOpeningPress in commands.js): a `--navi-open` button, a picker's
+ * press starts, with a `navi_open_press` delivered where the opening request
+ * would be (see announceOpeningPress in commands.js): a `--navi-open` button, a picker's
  * trigger when a press is what opens it, an expandable's UI part. Not the
  * anchor: it says where the popup is placed, not what opens it — a card
  * opened by a hold is pressed all day by taps meant for what it holds.
@@ -38701,42 +38704,36 @@ const usePopupContentMount = (
       anchorElement.removeEventListener("focusin", warm);
     };
   }, [contentMounted, anchor, mount]);
-  // Warm on an opening press, for "while-opened" (see the top comment).
-  // Listening for as long as the popup is there, not only while the content is
-  // unmounted: the press that throws a warmed content away can be a new
-  // opening press of this same popup, and it must warm again.
-  useEffect(() => {
-    if (mount !== "while-opened") {
-      return undefined;
-    }
-    const element = openController.getElement?.();
-    if (!element) {
-      return undefined;
-    }
-    let stopPressWarm = null;
-    const onOpeningPress = () => {
-      if (contentMountedRef.current || openController.onOpen) {
-        return;
-      }
-      const cancelBuild = requestFrameAfterNext(() => {
-        setContentMounted(true);
-      });
-      const stopWatching = watchPressEnd(() => {
-        stopPressWarm = null;
-        cancelBuild();
-        if (!openController.opened) {
-          setContentMounted(false);
+  // Warm on an opening press, for "while-opened" (see the top comment). Told
+  // by the popup's own element, which hears the announcement where it hears
+  // the opening request: `onnavi_open_press` beside each `onnavi_request_open`.
+  const stopPressWarmRef = useRef(null);
+  openController.onOpeningPress =
+    mount === "while-opened"
+      ? () => {
+          if (contentMountedRef.current || openController.onOpen) {
+            return;
+          }
+          const cancelBuild = requestFrameAfterNext(() => {
+            setContentMounted(true);
+          });
+          const stopWatching = watchPressEnd(() => {
+            stopPressWarmRef.current = null;
+            cancelBuild();
+            if (!openController.opened) {
+              setContentMounted(false);
+            }
+          });
+          stopPressWarmRef.current = () => {
+            stopPressWarmRef.current = null;
+            cancelBuild();
+            stopWatching();
+          };
         }
-      });
-      stopPressWarm = () => {
-        cancelBuild();
-        stopWatching();
-      };
-    };
-    element.addEventListener("navi_open_press", onOpeningPress);
+      : null;
+  useEffect(() => {
     return () => {
-      stopPressWarm?.();
-      element.removeEventListener("navi_open_press", onOpeningPress);
+      stopPressWarmRef.current?.();
     };
   }, [mount]);
 
@@ -52302,6 +52299,13 @@ const Expandable = props => {
     onnavi_request_open: e => {
       rest.onnavi_request_open?.(e);
       openController.open(e);
+    }
+    // The press announcing that request (see announceOpeningPress in
+    // commands.js), heard where the request is.
+    ,
+    onnavi_open_press: e => {
+      rest.onnavi_open_press?.(e);
+      openController.onOpeningPress?.();
     },
     onnavi_request_close: e => {
       rest.onnavi_request_close?.(e);
@@ -52381,6 +52385,9 @@ const ExpandableUI = ({
     onnavi_request_open: e => {
       openController.open(e);
     },
+    onnavi_open_press: () => {
+      openController.onOpeningPress?.();
+    },
     onnavi_request_close: e => {
       openController.requestClose(e, {
         isCancel: e.detail?.isCancel
@@ -64667,6 +64674,12 @@ const UncontrolledDialog = props => {
         // built (see open_controller.js).
         value: e.detail?.value
       });
+    }
+    // The press announcing that request (see announceOpeningPress in
+    // commands.js), heard where the request is.
+    ,
+    onnavi_open_press: () => {
+      openController.onOpeningPress?.();
     },
     onnavi_request_close: e => {
       const closing = openController.requestClose(e, {
@@ -66430,6 +66443,12 @@ const UncontrolledPopover = props => {
         // built (see open_controller.js).
         value: e.detail?.value
       });
+    }
+    // The press announcing that request (see announceOpeningPress in
+    // commands.js), heard where the request is.
+    ,
+    onnavi_open_press: () => {
+      openController.onOpeningPress?.();
     },
     onnavi_request_close: e => {
       const closing = openController.requestClose(e, {
@@ -67386,7 +67405,8 @@ const usePopoverProps = props => {
       onKeyDownShortcuts(e);
     },
     children
-    // onnavi_request_open/onnavi_request_close: for the uncontrolled case,
+    // onnavi_request_open/onnavi_request_close/onnavi_open_press: for the
+    // uncontrolled case,
     // already arrive here as plain props via ...rest (wired by
     // UncontrolledPopover above, forwarded through ControlledPopover's own
     // {...props} spread) — nothing extra to add here. A controlled caller
@@ -68257,6 +68277,12 @@ const PickerCustom = props => {
           }
         });
       },
+      // The press announcing that request (see announceOpeningPress in
+      // commands.js), heard where the request is — here too for the same
+      // reason. Nothing to gate: the announcer asked before saying it.
+      onnavi_open_press: () => {
+        openController.onOpeningPress?.();
+      },
       onnavi_request_close: e => {
         requestInteraction({
           event: e,
@@ -68729,6 +68755,7 @@ const PickerCalloutPopup = ({
   onnavi_request_open,
   onnavi_request_close,
   onnavi_request_confirm,
+  onnavi_open_press,
   mount,
   children: childrenProp
 }) => {
@@ -68821,6 +68848,7 @@ const PickerCalloutPopup = ({
       onnavi_request_open: onnavi_request_open,
       onnavi_request_close: onnavi_request_close,
       onnavi_request_confirm: onnavi_request_confirm,
+      onnavi_open_press: onnavi_open_press,
       children: jsx("div", {
         ref: hostRef,
         "data-picker-content": "",
@@ -77690,11 +77718,12 @@ const PickerButton = props => {
         // we re-dispatch on the input (to get escape to close for instance)
         ,
         onKeyDown: inputProps.onKeyDown
-        // in case request open/close are dispatched on the control root ->
-        // redispatch them to the host
+        // in case request open/close (or the press announcing an open) are
+        // dispatched on the control root -> redispatch them to the host
         ,
         onnavi_request_open: inputProps.onnavi_request_open,
-        onnavi_request_close: inputProps.onnavi_request_close
+        onnavi_request_close: inputProps.onnavi_request_close,
+        onnavi_open_press: inputProps.onnavi_open_press
         // `--navi-select`/`--navi-unselect` about one entry of the list the
         // picker holds — a chip on the façade, a suggestion beside the field.
         // Answered here rather than by the control drawing that list in the