@jsenv/navi 0.30.27 → 0.30.29

package/dist/dev/jsenv_navi.js

@@ -13987,7 +13987,7 @@ const CLICK_TO_EXPAND_SELECTOR = "summary, [aria-expanded]";
 // A popup is written inside whatever opened it, but it is not part of it on
 // screen: a control inside a popup must not be read as a click on the region
 // the popup happens to be nested in.
-const POPUP_SELECTOR = "[navi-control='popover'], [navi-control='dialog']";
+const POPUP_SELECTOR$1 = "[navi-control='popover'], [navi-control='dialog']";
 
 /**
  * Cancels `event` when the control consumed a click that a surrounding
@@ -14025,7 +14025,7 @@ const findClickToExpandRegion = (element) => {
   while (ancestor) {
     // Tested first: a popup carries `aria-expanded` of its own, so it would
     // otherwise pass for the region containing its own content.
-    if (ancestor.matches(POPUP_SELECTOR)) {
+    if (ancestor.matches(POPUP_SELECTOR$1)) {
       return null;
     }
     if (ancestor.matches(CLICK_TO_EXPAND_SELECTOR)) {
@@ -33521,6 +33521,16 @@ const useActionAsyncData = (action, {
   if (runningState === COMPLETED) {
     return [action.dataSignal.peek(), false, undefined];
   }
+  // Called off (abort(), a route left with nothing else asking for it): nothing
+  // is on its way until something runs it again. What it held stays drawn and
+  // no wait is claimed. A delegated wait holding nothing has nothing to draw,
+  // and suspends below until the next run.
+  if (runningState === ABORTED) {
+    const dataHeld = action.dataSignal.peek();
+    if (dataHeld !== undefined || loadingEffect === "use") {
+      return [dataHeld, false, undefined];
+    }
+  }
   if (runningState === FAILED) {
     if (prerun) {
       // Neither shown nor reported from a head start: the failure is the
@@ -33596,8 +33606,7 @@ const useActionAsyncData = (action, {
 
   // `loading: true` says the component draws its own wait, so this hook never
   // suspends: running, or holding params nobody has started yet — a debounced
-  // binding that just retargeted, an aborted run, a run this very render is
-  // about to issue. Either way what is on screen is not the current answer,
+  // binding that just retargeted, a run this very render is about to issue. Either way what is on screen is not the current answer,
   // which is what `loading` says. Suspending would not draw a spinner: it swaps
   // the subtree for the boundary, and everything under it remounts — a dialog
   // open inside it is then closed, by a hook the caller told to stay out of it.
@@ -33605,7 +33614,8 @@ const useActionAsyncData = (action, {
     return [action.dataSignal.peek(), true, undefined];
   }
 
-  // IDLE or RUNNING with loadingEffect: "delegate" — suspend
+  // IDLE, RUNNING, or called off holding nothing, with loadingEffect:
+  // "delegate" — suspend
   const reason = runningState === RUNNING ? "loading" : "idle";
   loadingRef.current = {
     reason,
@@ -33627,7 +33637,7 @@ const useActionAsyncData = (action, {
         } else if (reason === "loading" && state === ABORTED) {
           // The run waited on was called off: drawn again, this reader asks
           // for it again when nothing else will (calledOffBeforeThisReader),
-          // or waits for the next run as idle.
+          // draws what the instance held, or waits for the next run as idle.
           actionPendingPromiseWeakMap.delete(action);
           unsubscribe();
           resolve();
@@ -45038,6 +45048,17 @@ const resolveClosestExpandable = (el) => {
   }
   return expandableRepeatedBy(expandable) || expandable;
 };
+// The popup a decision taken inside it answers (a clear, a send). A section
+// revealed in the page flow (<Expandable>, data-in-flow) is not one: it belongs
+// to whatever holds it, a page or a popup, so the walk goes through it.
+const POPUP_SELECTOR = "[aria-expanded]:not([data-in-flow])";
+const resolveClosestPopup = (el) => {
+  const popup = el.closest(POPUP_SELECTOR);
+  if (!popup) {
+    return null;
+  }
+  return expandableRepeatedBy(popup) || popup;
+};
 // A control's host repeats the aria-expanded of its root, beside its role, for
 // assistive tech to read (a picker's <input>, a combobox or a button). The root
 // is the expandable: the host is a leaf the walk lands on only when it is the
@@ -45054,16 +45075,16 @@ const expandableRepeatedBy = (el) => {
   }
   return null;
 };
-// The surface a send is decided on: the expandable holding the CONTROL that
-// sends, never one that control is itself. A picker's own root carries
-// aria-expanded for its popup, and a confirm picker pressed as a whole (see
+// The surface a send is decided on: the popup holding the CONTROL that sends,
+// never one that control is itself. A picker's own root carries aria-expanded
+// for its popup, and a confirm picker pressed as a whole (see
 // picker_confirm.jsx) sends from its input: what it answers is the surface
 // around the picker, not the picker. Same for a popup told to send upward
 // once it has closed (data-after-send="--navi-send"): the surface above it.
-const resolveExpandableAround = (source) => {
+const resolvePopupAround = (source) => {
   const sender = findControlRoot(source) || source;
   const parent = sender.parentElement;
-  return parent ? resolveClosestExpandable(parent) : null;
+  return parent ? resolveClosestPopup(parent) : null;
 };
 // An element carrying navi-command-proxy-for="anchorId" stands in for that
 // other element as the command's source when the source is forwarded to
@@ -45089,16 +45110,14 @@ const resolveClosestControlWithAction = (el) => {
   return findClosestControlWithAction(el);
 };
 // Both can enclose the send's source: a form inside a dialog is a control with
-// an action, inside an expandable. The nearer one owns the send — a button
-// inside that form means "submit this form", not "confirm the dialog around
-// it", and the same button placed directly in the dialog means the opposite.
-const resolveClosestSendTarget = (expandable, controlWithAction) => {
-  if (!expandable || !controlWithAction) {
-    return expandable || controlWithAction;
+// an action, inside a popup. The nearer one owns the send — a button inside
+// that form means "submit this form", not "confirm the dialog around it", and
+// the same button placed directly in the dialog means the opposite.
+const resolveClosestSendTarget = (popup, controlWithAction) => {
+  if (!popup || !controlWithAction) {
+    return popup || controlWithAction;
   }
-  return expandable.contains(controlWithAction)
-    ? controlWithAction
-    : expandable;
+  return popup.contains(controlWithAction) ? controlWithAction : popup;
 };
 
 // An id that names something that is not a control — a layout box, a label, a
@@ -45300,9 +45319,10 @@ const clearCommand = (source, event) => {
     }
     // Only what is open: the clear cross of a picker sits on the closed
     // trigger, with nothing to close. A clear pressed INSIDE an open popup is
-    // the one this is for: it answers the popup, so the popup goes away.
-    const expandable = resolveClosestExpandable(source);
-    if (expandable?.getAttribute("aria-expanded") !== "true") {
+    // the one this is for: it answers the popup, so the popup goes away. An
+    // open section of the page is answered by nothing pressed in it.
+    const popup = resolveClosestPopup(source);
+    if (popup?.getAttribute("aria-expanded") !== "true") {
       return;
     }
     // Asked of the popup itself, not resolved again from the source: a
@@ -45310,7 +45330,7 @@ const clearCommand = (source, event) => {
     // picker names the input it empties), and a close resolved from it would
     // aim at that input's picker — busy with the send it was just handed —
     // rather than at the popup the cross is in.
-    triggerNaviCommand(expandable, "--navi-close", clearEvent, {
+    triggerNaviCommand(popup, "--navi-close", clearEvent, {
       optional: true,
     });
   };
@@ -45354,7 +45374,8 @@ const resetCommand = (source, event) => {
  *   same act, a level down. A popup opened from another popup answers for
  *   itself only — say `data-after-send="--navi-send"` on the sending control
  *   to make the decision travel up to the surface holding it;
- * - the document: nothing. A form on a page stays where it is.
+ * - the document: nothing. A form on a page stays where it is, open
+ *   <Expandable> sections around it included (see resolveClosestPopup).
  */
 // What follows a send that went through — and it is asked of the button that
 // sent before the form it sent, because a form with two submit buttons has two
@@ -45385,7 +45406,7 @@ const resolveAfterSend = (target, requester, surface) => {
 // the target: a popup that IS the send target is handled on its own (see the
 // aria-expanded branch in sendCommand), and must not answer twice.
 const resolveSendSurface = (target) =>
-  target.parentElement?.closest(`[data-slide], [aria-expanded]`);
+  target.parentElement?.closest(`[data-slide], ${POPUP_SELECTOR}`);
 // What follows a send is caused by the send, not by the gesture: the send
 // answered the gesture and may have prevented it (an Enter must not also click
 // the submit button, nor activate the trigger of the popup it closed), and a
@@ -45398,11 +45419,10 @@ const createSentEvent = (event) => {
 };
 
 const sendCommand = (source, event, { requester }) => {
-  const expandable = resolveExpandableAround(source);
   const target =
     resolveExplicitTarget(source) ||
     resolveClosestSendTarget(
-      expandable,
+      resolvePopupAround(source),
       resolveClosestControlWithAction(source),
     );
   if (!target) {
@@ -45413,7 +45433,7 @@ const sendCommand = (source, event, { requester }) => {
   // looking at, and a decision just taken there is a reason to leave it. The
   // NEAREST surface answers, and only that one — a form inside a slide inside a
   // dialog goes back a slide, it does not also close the dialog.
-  // send inside expandable — the control itself when it sends from its host,
+  // send inside popup — the control itself when it sends from its host,
   // whose aria-expanded is its root's (see expandableRepeatedBy)
   if (
     target.getAttribute("aria-expanded") === "true" &&
@@ -63046,7 +63066,11 @@ const Expandable = props => {
   return jsx(Box, {
     ref: rootRef,
     baseClassName: "navi_expandable",
-    "aria-expanded": opened ? "true" : "false",
+    "aria-expanded": opened ? "true" : "false"
+    // Read by commands.js: open or not, this is no popup — a clear or a send
+    // pressed inside does not close it (see resolveClosestPopup).
+    ,
+    "data-in-flow": "",
     "data-layout": isColumn ? "column" : undefined,
     "data-open-direction": revealDirection,
     "data-animation": animation ? "" : undefined,
@@ -63141,6 +63165,7 @@ const ExpandableUI = ({
     role: "button",
     tabIndex: 0,
     "aria-expanded": opened,
+    "data-in-flow": "",
     "aria-controls": contentId,
     onClick: onUIClick,
     onPointerDown: onUIPointerDown,