@jsenv/navi 0.29.375 → 0.29.377

package/dist/dev/jsenv_navi.js

@@ -35096,10 +35096,18 @@ registerNaviCommand("--navi-define", (source, event) => {
     implementation: () => executeNaviDefine(source, event, target),
   };
 });
+// A send whose target is the popup itself: what is around the source IS the
+// popup's answer, and a picker runs its action with that answer as it closes
+// (see onRequestClose in picker_custom.jsx) — dispatching the action here too
+// would run it twice. `isSend` tells this close from a dismissal (the cross,
+// a press outside, Escape): the person pressed a choice, which a picker may
+// act on even when the choice is the one it already held.
 const executeNaviDefine = (source, event, target) => {
-  // The picker's onClose already dispatches the action with the final value.
-  // Dispatching again here would fire the action twice.
-  return triggerNaviCommand(target, "--navi-close", event);
+  return dispatchCustomEvent(target, "navi_request_close", {
+    event,
+    source: resolveCommandProxySource(source),
+    isSend: true,
+  });
 };
 
 registerNaviCommand("--navi-scroll", (source, event) => {
@@ -66012,6 +66020,9 @@ const PickerCustom = props => {
     // closed on it. A confirm picker is the one saying something (see
     // picker_confirm.jsx): its press, deferred until the question is answered.
     onConfirm,
+    // A choice pressed inside the popup runs `action` even when it is the one
+    // the picker already held. See the JSDoc.
+    canSendWhileUnchanged,
     // The popup's lifecycle, the same pair Dialog and Popover take. Taken on
     // the picker and chained into its own openController below: the picker
     // hands the popup that controller, and a controlled Dialog/Popover reads
@@ -66054,6 +66065,7 @@ const PickerCustom = props => {
   delete pickerProps.defaultOpen;
   delete pickerProps.escapeEffect;
   delete pickerProps.onConfirm;
+  delete pickerProps.canSendWhileUnchanged;
   delete pickerProps.onOpen;
   delete pickerProps.onClose;
   delete pickerProps.openOn;
@@ -66155,8 +66167,17 @@ const PickerCustom = props => {
             // suggestion to accept, and confirming `undefined` cannot mean
             // anything. A picker on a defaultValue is untouched by this: it
             // shows something, so closing on it still confirms it.
-            // No action to run, but still allow the close.
-            return;
+            //
+            // Unless the close is a choice PRESSED inside the popup (a row's
+            // `--navi-send`, a `--navi-confirm`) on a picker that asked to
+            // send while unchanged: the person picked what was already picked,
+            // and for that picker the same value said again means something.
+            // A press outside, Escape, the cross say nothing and keep skipping.
+            const said = requestCloseEvent.detail.isSend || confirmEventRef.current;
+            if (!(canSendWhileUnchanged && said)) {
+              // No action to run, but still allow the close.
+              return;
+            }
           }
           dispatchRequestAction(inputEl, {
             event: requestCloseEvent,
@@ -66350,7 +66371,8 @@ const PickerCustom = props => {
           intent: "read",
           allowed: () => {
             const closing = requestClose(e, {
-              isCancel: e.detail.isCancel
+              isCancel: e.detail.isCancel,
+              isSend: e.detail.isSend
             });
             if (!closing) {
               e.preventDefault();
@@ -69979,8 +70001,17 @@ const useListScrollSync = ({
       // The list is holding itself somewhere; that is what owns the scroll.
       return;
     }
+    const scrollerEl = getScroller();
+    if (scrollerEl && (horizontal ? scrollerEl.scrollLeft : scrollerEl.scrollTop) === 0) {
+      // A scroller at its start has nothing above the view to keep still, and
+      // rows landing above the first one are what a list read from its start
+      // (a journal newest first, a feed) is there to show: holding the row on
+      // top would scroll past them. The browser's own scroll anchoring makes
+      // the same exception at offset 0.
+      return;
+    }
     anchorRef.current = captureScrollAnchor({
-      scrollerEl: getScroller(),
+      scrollerEl,
       listEl: getListEl(),
       items: listRows.visibleItemsSignal.peek(),
       horizontal
@@ -76597,6 +76628,15 @@ const PickerFirstResolver = props => {
  * @param {(confirmEvent: CustomEvent) => void} [onConfirm] Called once the
  *   popup has closed on a `--navi-confirm` said inside it. What a confirm
  *   picker uses to run its press; a picker of any type may listen too.
+ * @param {boolean} [canSendWhileUnchanged] Run `action` when a choice pressed
+ *   inside the popup — a row's `--navi-send`, a `--navi-confirm` — is the
+ *   value the picker already held. By default that close runs nothing: the
+ *   answer has not moved, so there is nothing new to say, same as `Form`.
+ *   Set this for a picker whose value is a setting that can be valid while
+ *   what it points to is not — a channel whose account came unlinked — so
+ *   picking it again is how the person asks for it to be put right, and the
+ *   action sees the same value and re-asks what there is to re-ask. A press
+ *   outside, Escape and the cross still send nothing: nothing was pressed.
  * @param {(openEvent: CustomEvent) => void} [onOpen] Called when the popup
  *   opens, whatever asked for it (a press, a command, history bringing it
  *   back). You most likely do NOT need this pair: "the user validated" is
@@ -87705,7 +87745,10 @@ const css$8 = /* css */`@layer navi {
       background-color: var(--x-fill-color);
       border-width: var(--border-width);
       border-radius: inherit;
-      clip-path: inset(0 calc((1 - var(--x-fill-ratio, 0)) * 100%) 0 0);
+      clip-path: inset(0
+            calc(100% - var(--border-width) - var(--x-fill-ratio, 0) *
+                (100% - var(--border-width) * 2))
+            0 0);
       border-style: solid;
       border-color: #0000;
       position: absolute;
@@ -87753,7 +87796,8 @@ const css$8 = /* css */`@layer navi {
 
   &[data-fill-round] {
     & .navi_meter_fill {
-      width: calc(var(--x-fill-ratio) * 100%);
+      width: calc(var(--x-fill-ratio) * (100% - var(--border-width) * 2) +
+            var(--border-width) * 2);
       clip-path: unset;
     }
   }