@jsenv/navi 0.29.323 → 0.29.325

package/dist/jsenv_navi.js

@@ -4,10 +4,10 @@
  */
 import { installImportMetaCssBuild, windowHeightSignal, windowWidthSignal, visualViewportHeightSignal, visualViewportWidthSignal, getAppHeight, getAppWidth, coarsePointerSignal, smallTouchScreenSignal } from "./jsenv_navi_side_effects.js";
 export { disableVirtualKeyboardOverlay } from "./jsenv_navi_side_effects.js";
-import { elementIsFocusable, createIterableWeakSet, dispatchInternalCustomEvent, dispatchCustomEvent, getVisuallyVisibleInfo, getFirstVisuallyVisibleAncestor, getElementSignature, createPubSub, findEvent, createValueEffect, findFocusDelegateTarget, findFocusable, scrollIntoViewThroughScrollables, allowWheelThrough, dispatchPublicCustomEvent, resolveCSSColor, ELEMENT_SIZE_CHANGE, findSelfOrAncestorFixedPosition, visibleRectEffect, pickPositionRelativeTo, getBorderSizes, getPaddingSizes, applyNewPosition, measureLongestVisualLineWidth, chainEvent, keepTouchRefusable, isPressDrivenClick, waitForTap, waitForPressHeld, suppressClickAfterGesture, startDragToTravel, dragSourceThatStoodDown, markDragSource, refuseDragTo, startDragTo, installPanZoom, getKeyboardEventDefaultAction, activeElementSignal, normalizeStyle, mergeOneStyle, getPositionedParent, normalizeStyles, createGroupTransitionController, getBorderRadius, preventIntermediateScrollbar, createOpacityTransition, watchWheelTravel, scrollRoomTowards, getScrollContainer, isTouchDrivenEvent, scrollIntoViewScoped, closestOpenableAncestor, isAncestorOpen, isDisplayedDespiteClosedAncestor, observeAncestorOpenState, getAncestorOpenType, findBefore, findAfter, resolveCSSSize, hasCSSSizeUnit, releaseWheelGesture, getScrollIntoViewScopedOffsets, wheelGestureIsTakenFrom, claimWheelGesture, initFocusGroup, stringifyStyle as stringifyStyle$1, resolveOklchLightness, contrastColor, parsePositionArea, snapToPixel, trapFocusInside, trapScrollInside, onAncestorReopen, isPressDisputedByDrag, canScroll, measureWidestChildRow, performTabNavigation, dragAfterIntent, stickyAsRelativeCoords, createDragToMoveGestureController, getDropTargetInfo, setStyles, useActiveElement } from "@jsenv/dom";
+import { elementIsFocusable, createIterableWeakSet, dispatchInternalCustomEvent, dispatchCustomEvent, getVisuallyVisibleInfo, getFirstVisuallyVisibleAncestor, getElementSignature, createPubSub, findEvent, createValueEffect, findFocusDelegateTarget, findFocusable, scrollIntoViewThroughScrollables, allowWheelThrough, dispatchPublicCustomEvent, resolveCSSColor, ELEMENT_SIZE_CHANGE, findSelfOrAncestorFixedPosition, visibleRectEffect, pickPositionRelativeTo, getBorderSizes, getPaddingSizes, applyNewPosition, measureLongestVisualLineWidth, chainEvent, keepTouchRefusable, isPressDrivenClick, waitForTap, waitForPressHeld, suppressClickAfterGesture, startDragToTravel, dragSourceThatStoodDown, markDragSource, refuseDragTo, startDragTo, installPanZoom, createInternalCustomEvent, getKeyboardEventDefaultAction, activeElementSignal, normalizeStyle, mergeOneStyle, getPositionedParent, normalizeStyles, createGroupTransitionController, getBorderRadius, preventIntermediateScrollbar, createOpacityTransition, watchWheelTravel, scrollRoomTowards, getScrollContainer, isTouchDrivenEvent, scrollIntoViewScoped, closestOpenableAncestor, isAncestorOpen, isDisplayedDespiteClosedAncestor, observeAncestorOpenState, getAncestorOpenType, findBefore, findAfter, resolveCSSSize, hasCSSSizeUnit, releaseWheelGesture, getScrollIntoViewScopedOffsets, wheelGestureIsTakenFrom, claimWheelGesture, initFocusGroup, stringifyStyle as stringifyStyle$1, resolveOklchLightness, contrastColor, parsePositionArea, snapToPixel, trapFocusInside, trapScrollInside, onAncestorReopen, isPressDisputedByDrag, canScroll, measureWidestChildRow, performTabNavigation, dragAfterIntent, stickyAsRelativeCoords, createDragToMoveGestureController, getDropTargetInfo, setStyles, useActiveElement } from "@jsenv/dom";
 export { chainEvent, clickIsSuppressed, contrastColor, createDragGestureController, dragAfterIntent, findEvent, markDragSource, startDragTo } from "@jsenv/dom";
 import { signal, computed, effect, untracked, batch, useComputed, useSignal } from "@preact/signals";
-import { isValidElement, createContext, render, h, toChildArray, options, cloneElement, Fragment as Fragment$1 } from "preact";
+import { isValidElement, createContext, render, h, toChildArray, options, cloneElement, createElement, Fragment as Fragment$1 } from "preact";
 import { useErrorBoundary, useLayoutEffect, useContext, useCallback, useRef, useState, useEffect, useMemo, useId } from "preact/hooks";
 import { humanizeI18n, setRuntimeLangSource, formatDuration, formatMonth, formatDay, resolveTimeRangePrecision, formatDatePlaceholder, toDate, getRelativeDay, formatDayRelative, formatMonthPlaceholder, formatWeekPlaceholder, formatDatetimePlaceholder, formatDatetime, toTimeOfDay, formatTimeOfDay, formatTime, formatMinuteDuration, formatSecondDuration, formatHourDuration, formatTimeRelative, formatNumber, interpolateText, installInterpolateJsx } from "@jsenv/humanize";
 export { createI18n, formatDatetime, formatDay, formatDayRelative, formatDuration, formatHourDuration, formatMinuteDuration, formatMonth, formatNumber, formatSecondDuration, formatTime, formatTimeOfDay, formatTimeRange, formatTimeRelative, interpolateText } from "@jsenv/humanize";
@@ -12616,6 +12616,12 @@ const useExecuteAction = (
     // whose command sent — is stored as the callout display target, so the
     // message appears on whoever asked rather than on the whole control.
     const element = elementRef.current;
+    if (!element) {
+      // The control left the page while its run was out (its own answer can
+      // unmount it, see the outcome events below): there is no box to hang
+      // the refusal on. onActionError is still told, by the run's side effect.
+      return;
+    }
     let target = requester;
     // A requester that is no longer on the page is not a place to show
     // anything: the clear cross leaves with the value it optimistically
@@ -12718,12 +12724,20 @@ const useExecuteAction = (
       // Either three callbacks, one per outcome, or a single one for "however
       // this ends" — it receives `{ aborted, reason }`, `{ error }` or
       // `{ data }`, so a caller that only needs to know the action settled
-      // (and whether it worked) does not have to register three.
+      // (and whether it worked) does not have to register three. Each also
+      // receives the outcome event (navi_action_abort/error/end) as its
+      // second argument.
       const addSideEffect = (sideEffect) => {
         if (typeof sideEffect === "function") {
-          addAbortCallback((reason) => sideEffect({ aborted: true, reason }));
-          addErrorCallback((error) => sideEffect({ error }));
-          addCompleteCallback((data) => sideEffect({ data }));
+          addAbortCallback((reason, outcomeEvent) =>
+            sideEffect({ aborted: true, reason }, outcomeEvent),
+          );
+          addErrorCallback((error, outcomeEvent) =>
+            sideEffect({ error }, outcomeEvent),
+          );
+          addCompleteCallback((data, outcomeEvent) =>
+            sideEffect({ data }, outcomeEvent),
+          );
           return;
         }
         const { abort, error, complete } = sideEffect;
@@ -12731,56 +12745,6 @@ const useExecuteAction = (
         addErrorCallback(error);
         addCompleteCallback(complete);
       };
-      addSideEffect({
-        abort: (reason) => {
-          const element = elementRef.current;
-          if (
-            // at this stage the action side effect might have removed the <element> from the DOM
-            // (in theory no because action side effect are batched to happen after)
-            // but other side effects might do this
-            element
-          ) {
-            dispatchInternalCustomEvent(element, "navi_action_abort", {
-              ...sharedActionEventDetail,
-              reason,
-            });
-          }
-        },
-        error: (error) => {
-          if (errorEffect === "show_validation_message") {
-            addErrorMessage(error, { requester });
-          } else if (errorEffect === "throw") {
-            setError(error);
-          }
-
-          const element = elementRef.current;
-          if (
-            // at this stage the action side effect might have removed the <element> from the DOM
-            // (in theory no because action side effect are batched to happen after)
-            // but other side effects might do this
-            element
-          ) {
-            dispatchInternalCustomEvent(element, "navi_action_error", {
-              ...sharedActionEventDetail,
-              error,
-            });
-          }
-        },
-        complete: (data) => {
-          const element = elementRef.current;
-          if (
-            // at this stage the action side effect might have removed the <element> from the DOM
-            // (in theory no because action side effect are batched to happen after)
-            // but other side effects might do this
-            element
-          ) {
-            dispatchInternalCustomEvent(element, "navi_action_end", {
-              ...sharedActionEventDetail,
-              data,
-            });
-          }
-        },
-      });
 
       const actionStartEventDetail = {
         ...sharedActionEventDetail,
@@ -12792,13 +12756,50 @@ const useExecuteAction = (
         actionStartEventDetail,
       );
 
+      // The outcome is told twice: to the element, when there still is one,
+      // and to the side effects registered at navi_action_start, always. A
+      // run can outlive its control — the run's own answer, written to a
+      // store the parent reads to decide what it draws, unmounts the control
+      // one microtask before the run settles — and the element is then no
+      // place to dispatch to: Preact drops a detached element's listeners.
+      // The side effects are the run's, not the element's, so the event
+      // reaches them either way.
+      const tellOutcome = (outcomeEventName, outcomeEventDetail, trigger) => {
+        const outcomeEvent = createInternalCustomEvent(outcomeEventName, {
+          ...sharedActionEventDetail,
+          ...outcomeEventDetail,
+        });
+        const element = elementRef.current;
+        if (element) {
+          element.dispatchEvent(outcomeEvent);
+        }
+        trigger(outcomeEvent);
+      };
+
       const runAction = () => {
         return action[method]({
           event: actionEvent,
           reason: `"${event.type}" event on ${getElementSignature(event.target)}`,
-          onAbort: triggerAbort,
-          onError: triggerError,
-          onComplete: triggerComplete,
+          onAbort: (reason) => {
+            tellOutcome("navi_action_abort", { reason }, (outcomeEvent) => {
+              triggerAbort(reason, outcomeEvent);
+            });
+          },
+          onError: (error) => {
+            if (errorEffect === "show_validation_message") {
+              addErrorMessage(error, { requester });
+            } else if (errorEffect === "throw") {
+              setError(error);
+            }
+            tellOutcome("navi_action_error", { error }, (outcomeEvent) => {
+              triggerError(error, outcomeEvent);
+            });
+          },
+          onComplete: (data) => {
+            tellOutcome("navi_action_end", { data }, (outcomeEvent) => {
+              triggerComplete(data, outcomeEvent);
+            });
+          },
         });
       };
 
@@ -31648,8 +31649,26 @@ const resolveExplicitTarget = (element) => {
   }
   return undefined;
 };
+// The control around the source that a command can be about: one holding a
+// value of its own. A popup is a control root too (it answers interactions,
+// carries a callout), but it has no host of its own — asked for one, it
+// answers with the first control INSIDE it, which for a field sending
+// --navi-update from a picker's popup is that very field: the command then
+// sets the field to what it already holds, which fires the command again,
+// without end. Such a surface is walked past, up to the picker or the form.
 const resolveFirstParentControl = (el) => {
-  return getParentControl(el);
+  let parentControl = getParentControl(el);
+  while (parentControl && !ownsControlHost(parentControl)) {
+    parentControl = getParentControl(parentControl);
+  }
+  return parentControl;
+};
+const ownsControlHost = (controlRoot) => {
+  const controlHost = findControlHost(controlRoot);
+  return (
+    Boolean(controlHost) &&
+    controlHost.closest("[navi-control]") === controlRoot
+  );
 };
 const resolveFirstChildControl = (el) => {
   let startEl;
@@ -34400,6 +34419,21 @@ const resolveAutoAnimationKind = (anchor, parsedPositionArea) => {
  */
 const OUTSIDE_REGION_ATTRIBUTE = "data-navi-popup-outside";
 
+/**
+ * `data-navi-popup-inside` is the other direction: a box of the PAGE the
+ * caller says belongs to a popup, so a press on it is not outside that popup
+ * even though it lies beyond the border box — a card whose press is what
+ * fills the one panel of a board. Its value names the popup by `id`, the way
+ * `commandfor` does (several ids, space separated), because the exemption is
+ * one popup's: a menu open beside the board still closes on the same press.
+ *
+ * Unlike the outside marker it answers for its whole subtree: the outside
+ * marker carves free space out of a surface whose controls stay surface, this
+ * one adds a whole thing — the card, its count, its buttons — to the popup's
+ * own ground, and nothing in it is a dismissal.
+ */
+const INSIDE_ATTRIBUTE = "data-navi-popup-inside";
+
 /**
  * What a press landing on a region the caller declared as not-its-surface
  * does: exactly what the same press on the backdrop would do.
@@ -34524,6 +34558,14 @@ const armOutsidePressClose = (
     ) {
       return;
     }
+    if (
+      popupEl.id &&
+      pointerDownEvent.target.closest?.(
+        `[${INSIDE_ATTRIBUTE}~="${CSS.escape(popupEl.id)}"]`,
+      )
+    ) {
+      return;
+    }
     // Real DOM containment wins over the rectangle below — an element
     // genuinely inside the popup (`overflow: visible`, a negative margin, an
     // absolutely-positioned child) can be painted outside its border box, and
@@ -39827,10 +39869,24 @@ const useInteractiveProps = (props, {
         // aborted run, whose promise is awaited to completion (see
         // performRun in actions.js) — which is exactly what "the server is
         // done with it" means, and therefore when the queued request may go.
-        e.detail.addSideEffect(outcome => {
+        e.detail.addSideEffect((outcome, outcomeEvent) => {
           uiStateController.actionInFlight = false;
           uiStateController.runningActionSignal.value = null;
           uiStateController.parallelGuard?.release(uiStateController);
+          // The outcome callbacks are told from here, the run's side effect,
+          // and not from the onnavi_action_* handlers below: the run can
+          // outlive this control (its own answer, written to a store the
+          // parent reads, unmounts it one microtask before it settles), and
+          // whoever wears the run — a parent's `loading` following these,
+          // see 12_picker_card_demo.html — must hear it end all the same.
+          // The handlers below keep what needs the element.
+          if (outcome.aborted) {
+            onActionAborted?.(outcomeEvent);
+          } else if ("error" in outcome) {
+            onActionError?.(outcome.error, outcomeEvent);
+          } else {
+            onActionEnd?.(outcome.data, outcomeEvent);
+          }
           const queuedEvent = uiStateController.queuedActionAllowedEvent;
           if (!queuedEvent) {
             syncOptimisticAttribute(false);
@@ -39870,7 +39926,6 @@ const useInteractiveProps = (props, {
         if (resetOnAbort && !superseded) {
           dispatchRequestResetUIState(e.currentTarget, e);
         }
-        onActionAborted?.(e);
       },
       onnavi_action_error: e => {
         const {
@@ -39879,7 +39934,6 @@ const useInteractiveProps = (props, {
         if (resetOnError) {
           dispatchRequestResetUIState(e.currentTarget, e);
         }
-        onActionError?.(error, e);
         uiStateController.onActionError(e);
       },
       onnavi_action_end: e => {
@@ -39887,7 +39941,6 @@ const useInteractiveProps = (props, {
           data
         } = e.detail;
         debugAction(e, `action end with data: ${JSON.stringify(data)}`);
-        onActionEnd?.(data, e);
         controlRootProps.onnavi_action_end?.(e);
         uiStateController.onActionEnd(e);
 
@@ -47792,33 +47845,6 @@ const createToggleEvent = open => {
   return toggleEvent;
 };
 
-const NextResolverContext = createContext(null);
-const useNextResolver = () => useContext(NextResolverContext);
-
-/**
- * Creates a renderComponent function that passes props through a chain of resolvers.
- * Each resolver is a Preact component rendered in sequence (hooks are allowed).
- * To pass through to the next resolver, call useNextResolver() and render the
- * returned Next component with the desired props.
- * To terminate the chain early (e.g. render a specialized component), render
- * directly without calling Next.
- *
- * The last entry in the array is the final/target component — it receives null
- * from useNextResolver() indicating it is terminal.
- *
- * Usage:
- *   const renderButton = createComponentResolver([ResolverA, ResolverB, ButtonTarget]);
- *   // Then inside a component render:
- *   renderButton(props)
- *
- * Each position of the chain has a runner of its own, defined once: it renders
- * its resolver under a NextResolverContext holding the runner after it. A
- * resolver that re-renders on its own and renders <Next> therefore resumes at
- * its own position without any bookkeeping — the Next it was given is the one
- * made for it. A chain rendered by the hundred (a list's rows) pays two
- * components per position, the runner and the resolver, so nothing here is a
- * component that merely forwards.
- */
 const createComponentResolver = (resolvers, {
   pure
 } = {}) => {
@@ -47827,13 +47853,10 @@ const createComponentResolver = (resolvers, {
   for (let index = 0; index < resolvers.length; index++) {
     const Resolver = resolvers[index];
     const isLast = index === lastIndex;
-    const Runner = props => jsx(NextResolverContext.Provider, {
-      value: isLast ? null : runners[index + 1],
-      children: jsx(Resolver, {
-        ...props
-      })
-    });
-    Runner.displayName = `${Resolver.displayName || Resolver.name}Runner`;
+    function Runner(props) {
+      return runResolverBody(this, Resolver, props, isLast ? null : runners[index + 1]);
+    }
+    Runner.displayName = Resolver.displayName || Resolver.name;
     runners.push(Runner);
   }
   const FirstRunner = runners[0];
@@ -47864,6 +47887,54 @@ const createComponentResolver = (resolvers, {
   }
   return PureRenderComponent;
 };
+const useNextResolver = () => {
+  if (currentNext === OUTSIDE_RESOLVER_BODY) {
+    throw new Error("useNextResolver() called outside a resolver body: a resolver rendered by another resolver is rendered with renderResolver(Resolver, props), not as a JSX child.");
+  }
+  return currentNext;
+};
+const renderResolver = (Resolver, props) => {
+  if (currentNext === OUTSIDE_RESOLVER_BODY) {
+    throw new Error(`renderResolver(${Resolver.displayName || Resolver.name}) called outside a resolver body: only a resolver can render another one.`);
+  }
+  let SubRunner = subRunnerByResolver.get(Resolver);
+  if (!SubRunner) {
+    SubRunner = function (props) {
+      const {
+        [NEXT_PROP]: next,
+        ...ownProps
+      } = props;
+      return runResolverBody(this, Resolver, ownProps, next);
+    };
+    SubRunner.displayName = Resolver.displayName || Resolver.name;
+    subRunnerByResolver.set(Resolver, SubRunner);
+  }
+  return createElement(SubRunner, {
+    ...props,
+    [NEXT_PROP]: currentNext
+  });
+};
+
+// What useNextResolver() answers: the runner after the one whose body is
+// running, null for the last — and, outside any body, a value that is neither.
+const OUTSIDE_RESOLVER_BODY = Symbol("outside_resolver_body");
+let currentNext = OUTSIDE_RESOLVER_BODY;
+const runResolverBody = (instance, Resolver, props, next) => {
+  const nextBefore = currentNext;
+  currentNext = next;
+  try {
+    return Resolver.call(instance, props);
+  } finally {
+    currentNext = nextBefore;
+  }
+};
+
+// One runner per resolver rendered through renderResolver, made on first use:
+// the same component type on every render, so a resolver picked again is
+// updated in place and another one picked is a remount, like any child.
+const subRunnerByResolver = new WeakMap();
+// Rides on the props of a sub-runner, stripped before the body sees them.
+const NEXT_PROP = "navi-resolver-next";
 function pureShouldComponentUpdate(nextProps) {
   return shallowDiffers(this.props, nextProps);
 }
@@ -48689,9 +48760,7 @@ const ButtonConfirm = ({
 const ButtonRouteResolver = props => {
   const Next = useNextResolver();
   if (props.route) {
-    return jsx(ButtonWithRoute, {
-      ...props
-    });
+    return renderResolver(ButtonWithRoute, props);
   }
   return jsx(Next, {
     ...props
@@ -51289,9 +51358,7 @@ const installInputCss = () => {
 const InputModeResolver = props => {
   const Next = useNextResolver();
   if (props.inputMode === "numeric" || props.inputMode === "decimal") {
-    return jsx(InputModeNumericOrDecimal, {
-      ...props
-    });
+    return renderResolver(InputModeNumericOrDecimal, props);
   }
   return jsx(Next, {
     ...props
@@ -51519,39 +51586,25 @@ const InputTypeResolver = props => {
   // very same "icon while empty, clear button once filled" affordance without
   // pretending to be a search box.
   if (props.clearable && props.type !== "search") {
-    return jsx(InputClearable, {
-      ...props
-    });
+    return renderResolver(InputClearable, props);
   }
   if (props.type === "search") {
-    return jsx(InputSearch, {
-      ...props
-    });
+    return renderResolver(InputSearch, props);
   }
   if (props.type === "email") {
-    return jsx(InputEmail, {
-      ...props
-    });
+    return renderResolver(InputEmail, props);
   }
   if (props.type === "tel") {
-    return jsx(InputTel, {
-      ...props
-    });
+    return renderResolver(InputTel, props);
   }
   if (props.type === "number") {
-    return jsx(InputNumber, {
-      ...props
-    });
+    return renderResolver(InputNumber, props);
   }
   if (props.type === "color") {
-    return jsx(InputColor, {
-      ...props
-    });
+    return renderResolver(InputColor, props);
   }
   if (props.type === "datetime-local") {
-    return jsx(InputDatetimeLocal, {
-      ...props
-    });
+    return renderResolver(InputDatetimeLocal, props);
   }
   return jsx(Next, {
     ...props
@@ -51664,9 +51717,7 @@ const InputDatetimeLocal = props => {
 const InputWithListResolver = props => {
   const Next = useNextResolver();
   if (props["navi-list"]) {
-    return jsx(InputWithList, {
-      ...props
-    });
+    return renderResolver(InputWithList, props);
   }
   return jsx(Next, {
     ...props
@@ -51773,9 +51824,7 @@ const ChevronDownSvg$1 = () => {
 const InputWithSuggestionsResolver = props => {
   const Next = useNextResolver();
   if (props["navi-suggestions"]) {
-    return jsx(InputTextualWithSuggestions, {
-      ...props
-    });
+    return renderResolver(InputTextualWithSuggestions, props);
   }
   return jsx(Next, {
     ...props
@@ -59279,34 +59328,44 @@ const css$E = /* css */`
  *   `showModal()`'d (native focus trap, `Escape`-to-cancel, hardware
  *   back-button dismissal, rest-of-document made inert), without one it is
  *   shown through the Popover API and the page behind stays live.
- * @param {boolean} [props.dockedOnSmallTouchScreen] - Turns the dialog into a
- *   bottom sheet (docked flush to the bottom edge, full width) on a small touch
- *   screen, and leaves it alone otherwise. For a dialog meant to be interacted
- *   with rather than merely read: on a phone the keyboard owns the bottom of
- *   the screen and a centered box ends up both cramped and out of thumb reach,
- *   while under a mouse the centered box is already the right shape. Both
- *   halves of the name matter (`smallTouchScreenSignal`): touch alone would
- *   dock a big touch screen — a tablet, a kiosk panel — a whole screen away
- *   from where the finger just tapped, and size alone would dock a narrow
- *   desktop window, which is still a mouse. It supplies defaults for
- *   `positionArea`, `marginWithContainer`, `expandX` and `scrollCapture`, so
- *   any of them can still be pinned explicitly — including `expandX={false}`,
+ * @param {boolean|"top"|"bottom"} [props.dockedOnSmallTouchScreen] - Turns
+ *   the dialog into a sheet (docked flush to one edge, full width) on a small
+ *   touch screen, and leaves it alone otherwise: under a mouse the centered box
+ *   is already the right shape, while on a phone a centered box ends up cramped
+ *   between the keyboard and the top of the screen. The value says which edge
+ *   the sheet rests on. `true` (or `"top"`) rests it on the top edge, out of
+ *   the virtual keyboard's way: a field tapped in the sheet raises the keyboard
+ *   below it, and the question and the field stay where the finger found them.
+ *   `"bottom"` rests it on the bottom edge, where the thumbs are — for a sheet
+ *   that is read and tapped (a list, a couple of buttons, a confirmation) and
+ *   holds nothing that raises a keyboard; one that does is reflowed into the
+ *   strip left above the keyboard at every keystroke. The edge is said here
+ *   rather than through `positionArea`, which places both shapes:
+ *   `positionArea="bottom"` would also pin the centered box to the bottom of a
+ *   desktop window. Both halves of the name matter (`smallTouchScreenSignal`):
+ *   touch alone would dock a big touch screen — a tablet, a kiosk panel — a
+ *   whole screen away from where the finger just tapped, and size alone would
+ *   dock a narrow desktop window, which is still a mouse. It supplies defaults
+ *   for `positionArea`, `marginWithContainer`, `expandX` and `scrollCapture`,
+ *   so any of them can still be pinned explicitly — including `expandX={false}`,
  *   which opts the docked dialog out of the full-width stretch and leaves it a
- *   floating box at the bottom. It also withdraws `maxWidth` while docked: a
+ *   floating box against its edge. It also withdraws `maxWidth` while docked: a
  *   sheet is container-wide by definition, and a `maxWidth` is an answer about
  *   the *centered* shape, so the two can be stated together (`maxWidth="16rem"
  *   dockedOnSmallTouchScreen`) and each applies where it means something.
  *   `minWidth` needs no such rule — its floor is below the full width — and
  *   `maxHeight`/`minHeight` keep applying, a sheet being content-tall.
- *   Ignored entirely when `expandY`
- *   (or `expand`) is set: a dialog already filling the height is on the bottom
- *   edge docking would bring it to, so docking could only take away the shape
- *   the caller asked for. Re-resolves live as the pointer
- *   type or the window size changes. A sheet resting on the bottom edge is also
- *   pushed back down to close it, held by its header (a `Box` with the `header`
- *   prop) and by anything else carrying `data-swipe-grip`. The rest of the sheet
- *   is left to what it holds, so a board something is dragged across keeps its
- *   own gestures. See `swipe_to_close.js`.
+ *   Ignored entirely when `expandY` (or `expand`) is set: a dialog already
+ *   filling the height touches both edges, so docking could only take away the
+ *   shape the caller asked for. Re-resolves live as the pointer type or the
+ *   window size changes. A sheet is also pushed back through the edge it rests
+ *   on to close it (down for a bottom sheet, up for a top one), held by its
+ *   header (a `Box` with the `header` prop) and by anything else carrying
+ *   `data-swipe-grip`. The rest of the sheet is left to what it holds, so a
+ *   board something is dragged across keeps its own gestures. See
+ *   `swipe_to_close.js`. In dev, a `"bottom"` sheet found holding a field that
+ *   raises the keyboard is warned about at open, on any screen — the mistake
+ *   itself only shows on a phone.
  * @param {string} [props.positionArea="center"] - Where to dock the dialog
  *   within its container (the viewport for `layer="top"`, the positioned
  *   ancestor for `layer="local"`) — Dialog is never anchored to a real
@@ -59672,28 +59731,28 @@ const DialogLocal = props => {
  * own to render: a modal's is the native `::backdrop`, and `backdrop={false}`
  * has none at all.
  */
-// What a dialog turns into on a small touch screen. "bottom" is not a taste:
-// it puts the dialog in the zone a phone is actually operated from — where the
-// thumbs rest and where the virtual keyboard comes up — instead of the middle
-// of the screen, which is the farthest point from both.
+// What a dialog turns into on a small touch screen: a sheet flush against one
+// edge of the screen. Which edge is the caller's word (the prop's value, see
+// its JSDoc), the top one unless told "bottom": the cheap mistake must be the
+// default one — a sheet docked top by mistake is a little farther from the
+// thumb, a sheet docked bottom by mistake fights the virtual keyboard, which
+// rises from that very edge, on every keystroke.
 // Only defaults: an explicitly passed prop still wins, so the docked shape can
 // be adjusted one axis at a time instead of being all-or-nothing.
 const DOCKED = {
-  positionArea: "bottom",
   marginWithContainer: 0,
   expandX: true,
-  // A sheet resting on the bottom edge is dragged with a thumb, and a drag that
-  // runs past its own edge must not land on the page behind it: the same
-  // reasoning as "bottom" above, applied to the gesture instead of the shape.
+  // A sheet is dragged with a thumb, and a drag that runs past its own edge
+  // must not land on the page behind it.
   scrollCapture: true
 };
 
-// Where a bottom sheet is held to push it back down: the strip a Box declares
-// with `header`, plus anything the application marked as one more. Everything
-// else in the sheet is content the finger came to operate — a board a piece is
-// dragged across, a list, a map — and a press there belongs to it. A sheet with
-// no header and nothing marked is not pushed down at all; it is closed by its
-// own controls, by the backdrop and by Escape.
+// Where a sheet is held to push it back through its edge: the strip a Box
+// declares with `header`, plus anything the application marked as one more.
+// Everything else in the sheet is content the finger came to operate — a board
+// a piece is dragged across, a list, a map — and a press there belongs to it. A
+// sheet with no header and nothing marked is not pushed back at all; it is
+// closed by its own controls, by the backdrop and by Escape.
 const DOCKED_SWIPE_GRIP = "[data-header],[data-swipe-grip]";
 const useDialogProps = props => {
   const backdropProps = {};
@@ -59799,12 +59858,12 @@ const useDialogProps = props => {
   // Only a small touch screen changes anything: on a mouse — and on a touch
   // screen too big to reach the bottom edge of — a dialog already wants to be
   // the centered box it is by default, so there is nothing to resolve.
-  // expandY cancels the docking outright: docking exists to bring the dialog
-  // down to the edge the thumb is on, and a dialog already filling the height
-  // is on that edge — all docking could still do is take away the shape the
-  // caller asked for (and arm a swipe-down on something that never rose).
+  // expandY cancels the docking outright: a dialog already filling the height
+  // touches both edges — all docking could still do is take away the shape
+  // the caller asked for (and arm a swipe on something that never rose).
   const isDocked = dockedOnSmallTouchScreen && smallTouchScreenSignal.value && !expandY;
-  const positionArea = positionAreaProp ?? (isDocked ? DOCKED.positionArea : "center");
+  const dockedEdge = dockedOnSmallTouchScreen === "bottom" ? "bottom" : "top";
+  const positionArea = positionAreaProp ?? (isDocked ? dockedEdge : "center");
   const marginWithContainer = marginWithContainerProp ?? (isDocked ? DOCKED.marginWithContainer :
   // A share of whatever holds the dialog: the app's own screen for a
   // top-layer one — where appw is exactly "3% of the container", the
@@ -59866,12 +59925,12 @@ const useDialogProps = props => {
     y: "center",
     x: "center"
   };
-  // Pushing the sheet back down closes it — a bottom sheet is reached with a
-  // thumb, and the thumb is already on the edge it would push. Only a sheet
-  // actually resting on the bottom edge: anywhere else the gesture would send
-  // the dialog somewhere it never came from.
-  const swipeToCloseDown = isDocked && parsedPositionArea.y === "bottom";
-  const onSwipePointerDown = swipeToCloseDown ? createSwipeToClose("bottom", {
+  // Pushing the sheet back through the edge it rests on closes it — the way
+  // it came in, and the way the thumb on that edge already pushes. Only a sheet
+  // actually resting on an edge: anywhere else the gesture would send the
+  // dialog somewhere it never came from.
+  const swipeToCloseSide = isDocked && (parsedPositionArea.y === "top" || parsedPositionArea.y === "bottom") ? parsedPositionArea.y : null;
+  const onSwipePointerDown = swipeToCloseSide ? createSwipeToClose(swipeToCloseSide, {
     grip: DOCKED_SWIPE_GRIP
   }) : null;
   // A corner sitting exactly on the container's own corner must not be
@@ -60503,12 +60562,12 @@ const useDialogProps = props => {
     "data-flush-right": flushEdges.right ? "" : undefined,
     "data-flush-bottom": flushEdges.bottom ? "" : undefined,
     "data-flush-left": flushEdges.left ? "" : undefined,
-    "data-swipe-to-close": swipeToCloseDown ? "" : undefined,
+    "data-swipe-to-close": swipeToCloseSide ?? undefined,
     // The axis the sheet travels on when it is pushed back, said to the shared
     // gesture layer: it is what a box travelling inside the sheet reads to know
     // this axis is already walked (see @jsenv/dom's drag_to_travel).
-    "data-drag-travel": swipeToCloseDown ? SWIPE_AXIS_BY_SIDE.bottom : undefined,
-    "data-travel-by-drag": swipeToCloseDown ? SWIPE_AXIS_BY_SIDE.bottom : undefined,
+    "data-drag-travel": swipeToCloseSide ? SWIPE_AXIS_BY_SIDE[swipeToCloseSide] : undefined,
+    "data-travel-by-drag": swipeToCloseSide ? SWIPE_AXIS_BY_SIDE[swipeToCloseSide] : undefined,
     "onPointerDown": e => {
       rest.onPointerDown?.(e);
       onSwipePointerDown?.(e);
@@ -62302,8 +62361,8 @@ const css$C = /* css */`@layer navi {
  * @param {string} [props.positionAreaFixed] - **Popover-only**, same guard.
  * @param {string} [props.positionAreaWhenAnchorIsInvalid] - **Popover-only**,
  *   same guard.
- * @param {boolean} [props.dockedOnSmallTouchScreen] - **Dialog-only** (a
- *   popover is never a bottom sheet), destructured out for the same reason
+ * @param {boolean|"top"|"bottom"} [props.dockedOnSmallTouchScreen] -
+ *   **Dialog-only** (a popover is never a sheet), destructured out for the same reason
  *   turned around: it must not land on the popover element as a stray DOM
  *   attribute when the screen-size resolution picks `mode="popover"`. Where
  *   that resolution lands is what decides whether it applies at all, so a
@@ -62331,7 +62390,7 @@ const css$C = /* css */`@layer navi {
  *   honoured in either mode — a wall-less `Dialog` is shown through the
  *   Popover API rather than `showModal()` — so which mode the screen-size
  *   resolution picks says nothing about whether the page behind stays live.
- *   It is also what makes a sheet docked to the bottom of a phone's screen
+ *   It is also what makes a sheet docked to an edge of a phone's screen
  *   (`dockedOnSmallTouchScreen`) non-modal.
  * @param {"auto"|"discrete"|"invisible"} [props.backdropVariant] - Forwarded
  *   as-is to whichever component renders (both understand it identically):
@@ -62527,9 +62586,7 @@ const css$B = /* css */`.navi_picker_callout_dock > [data-picker-content] {
 const PickerCustomResolver = props => {
   import.meta.css = [css$B, "@jsenv/navi/src/control/picker/picker_custom.jsx"];
   if (props.children === undefined) {
-    return jsx(PickerNative, {
-      ...props
-    });
+    return renderResolver(PickerNative, props);
   }
   if (props.mode === "callout") {
     // A tooltip is an icon one presses, unless told otherwise. Own-property
@@ -62591,14 +62648,12 @@ const PickerCustomResolver = props => {
     // how a field says its value is a JS one, kept beside the DOM (see
     // controller_registry.js) — the same thing type="array"/"object" already
     // say for their shapes.
-    return jsx(PickerCustom, {
+    return renderResolver(PickerCustom, {
       ...props,
       type: "navi_js"
     });
   }
-  return jsx(PickerCustom, {
-    ...props
-  });
+  return renderResolver(PickerCustom, props);
 };
 const PickerNative = props => {
   const Next = useNextResolver();
@@ -63210,9 +63265,9 @@ const PickerCustom = props => {
       });
     }
   }
-  return jsx(PickerContentInsidePopup, {
+  return renderResolver(PickerContentInsidePopup, {
     ...pickerProps,
-    mode: mode
+    mode
   });
 };
 
@@ -64479,14 +64534,10 @@ const createItemTracker = (onChange) => {
 const ListItemHeaderOrFooterResolver = props => {
   const Next = useNextResolver();
   if (props.header) {
-    return jsx(ListItemHeader, {
-      ...props
-    });
+    return renderResolver(ListItemHeader, props);
   }
   if (props.footer) {
-    return jsx(ListItemFooter, {
-      ...props
-    });
+    return renderResolver(ListItemFooter, props);
   }
   return jsx(Next, {
     ...props
@@ -64718,9 +64769,7 @@ const ListSelectableContext = createContext(false);
 const ListSelectableResolver = props => {
   const Next = useNextResolver();
   if (props.selectable) {
-    return jsx(ListSelectable, {
-      ...props
-    });
+    return renderResolver(ListSelectable, props);
   }
   return jsx(ListSelectableContext.Provider, {
     value: false,
@@ -65020,7 +65069,7 @@ const ListItemSelectableResolver = props => {
   const isHeaderOrFooter = Boolean(props.header || props.footer);
   const selectable = props.selectable ?? (isHeaderOrFooter || props.role === "presentation" ? false : listSelectable);
   if (selectable) {
-    return jsx(ListItemSelectable, {
+    return renderResolver(ListItemSelectable, {
       ...props,
       selectable: true
     });
@@ -70263,14 +70312,10 @@ const generateTimeSlots = (min, max, stepSeconds) => {
 const PickerPresetResolver = props => {
   const Next = useNextResolver();
   if (props.type === "navi_time") {
-    return jsx(PickerNaviTime, {
-      ...props
-    });
+    return renderResolver(PickerNaviTime, props);
   }
   if (props.type === "navi_minute") {
-    return jsx(PickerNaviMinute, {
-      ...props
-    });
+    return renderResolver(PickerNaviMinute, props);
   }
   return jsx(Next, {
     ...props
@@ -71068,59 +71113,37 @@ const PencilSvg = () => {
 const PickerTypeResolver = props => {
   const Next = useNextResolver();
   if (props.type === "color") {
-    return jsx(PickerColor, {
-      ...props
-    });
+    return renderResolver(PickerColor, props);
   }
   if (props.type === "datetime") {
-    return jsx(PickerDatetime, {
-      ...props
-    });
+    return renderResolver(PickerDatetime, props);
   }
   if (props.type === "date") {
-    return jsx(PickerDate, {
-      ...props
-    });
+    return renderResolver(PickerDate, props);
   }
   if (props.type === "month") {
-    return jsx(PickerMonth, {
-      ...props
-    });
+    return renderResolver(PickerMonth, props);
   }
   if (props.type === "week") {
-    return jsx(PickerWeek, {
-      ...props
-    });
+    return renderResolver(PickerWeek, props);
   }
   if (props.type === "time") {
-    return jsx(PickerTime, {
-      ...props
-    });
+    return renderResolver(PickerTime, props);
   }
   if (props.type === "duration") {
-    return jsx(PickerDuration, {
-      ...props
-    });
+    return renderResolver(PickerDuration, props);
   }
   if (props.type === "file") {
-    return jsx(PickerFile, {
-      ...props
-    });
+    return renderResolver(PickerFile, props);
   }
   if (props.type === "text") {
-    return jsx(PickerText, {
-      ...props
-    });
+    return renderResolver(PickerText, props);
   }
   if (props.type === "array") {
-    return jsx(PickerArray, {
-      ...props
-    });
+    return renderResolver(PickerArray, props);
   }
   if (props.type === "object") {
-    return jsx(PickerObject, {
-      ...props
-    });
+    return renderResolver(PickerObject, props);
   }
   return jsx(Next, {
     ...props
@@ -74972,7 +74995,7 @@ const css$n = /* css */`.navi_split_button {
  *   dialogExpand?: boolean,
  *   dialogExpandX?: boolean,
  *   dialogExpandY?: boolean,
- *   dockedOnSmallTouchScreen?: boolean,
+ *   dockedOnSmallTouchScreen?: boolean | "top" | "bottom",
  *   marginWithContainer?: number | string,
  *   backdrop?: boolean,
  *   backdropVariant?: "auto" | "discrete" | "invisible",
@@ -84249,7 +84272,9 @@ const css = /* css */`.navi_side_panel {
  *   closes the panel on an outside click instead, and also enables trapping
  *   Tab navigation inside the panel (`focusCapture`) — closing on outside
  *   interaction only makes sense paired with not letting focus silently
- *   leave the panel first.
+ *   leave the panel first. A box of the page whose press must not close the
+ *   panel (a card that fills it) names the panel:
+ *   `data-navi-popup-inside={id}` — see docs/popup_backdrop.md.
  * @param {boolean} [props.swipeToClose=true] - Pushing the panel back
  *   towards the edge it is docked to closes it: the panel follows the
  *   pointer and finishes leaving (or comes back to rest) when it is
@@ -84384,6 +84409,7 @@ SidePanel.Foot = SidePanelFoot;
 
 const createSlot = (SlotRenderer = Box) => {
   const slotPropsSignal = signal();
+  let filler = null;
   const Slot = () => {
     const props = slotPropsSignal.value;
     return jsx(SlotRenderer, {
@@ -84392,9 +84418,18 @@ const createSlot = (SlotRenderer = Box) => {
     });
   };
   const SlotFill = props => {
+    const fillerRef = useRef();
+    filler = fillerRef;
     slotPropsSignal.value = props;
     useLayoutEffect(() => {
       return () => {
+        // Within one diff preact mounts the newcomer before unmounting the
+        // leaver, so the leaver may no longer be the filler by the time it
+        // is cleaned up.
+        if (filler !== fillerRef) {
+          return;
+        }
+        filler = null;
         slotPropsSignal.value = null;
       };
     }, []);