@jsenv/navi 0.29.376 → 0.29.378

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) => {
@@ -45310,6 +45318,10 @@ time.navi_text {
     border-radius: inherit;
   }
 
+  &[data-text-clamp] {
+    min-width: 0;
+  }
+
   &[data-text-overflow] {
     text-overflow: ellipsis;
     overflow-wrap: normal;
@@ -45683,6 +45695,10 @@ const shouldInjectSpacingBetween = (left, right) => {
  *   n lines. This is the only prop to use for that — `Box`'s `lineClamp` /
  *   `overflowEllipsis` are raw CSS mappings meant for elements that are not a
  *   `Text`, and `lineClamp={1}` is never the single-line truncation you want.
+ *   Either value states the `white-space` it needs — one line does not wrap,
+ *   n lines may — so it holds inside a single-line ancestor too (a `Picker`'s
+ *   value, a `Time`); an explicit `noWrap`/`pre`/`preLine` on this `Text`
+ *   still wins.
  *   Truncation only happens if the element may become narrower than its
  *   content: `maxLines` sets `min-width: 0` here, but each `Box` between this
  *   one and the element that carries the width must set it too.
@@ -45745,6 +45761,25 @@ const shouldInjectSpacingBetween = (left, right) => {
  *   internally for overlays such as the skeleton container.
  */
 const Text = props => {
+  if (props.maxLines > 1) {
+    props = {
+      ...props,
+      "data-text-clamp": ""
+    };
+    if (!saysWhiteSpace(props)) {
+      // n lines means "you may wrap". white-space is inherited, so left unsaid
+      // a single-line ancestor (a Picker's value, a Time) leaves the clamp one
+      // line to cut, and the text runs past its box. Same tag rule as
+      // TextOverflow: a paragraph keeps its line breaks.
+      props = props.as === "p" ? {
+        ...props,
+        preLine: true
+      } : {
+        ...props,
+        noWrap: false
+      };
+    }
+  }
   if (props.loading || props.skeleton) {
     return jsx(TextSkeleton, {
       ...props
@@ -45956,6 +45991,14 @@ const TextOverflow = ({
     children: children
   });
 };
+const saysWhiteSpace = ({
+  noWrap,
+  pre,
+  preWrap,
+  preLine
+}) => {
+  return noWrap !== undefined || pre !== undefined || preWrap !== undefined || preLine !== undefined;
+};
 const TextWithSelectRange = ({
   ref,
   selectRange,
@@ -59993,9 +60036,11 @@ const TimeRelative = ({
   });
 };
 const TimeText = props => {
+  // A date stays on one line, unless the caller grants it several.
+  const noWrap = props.maxLines > 1 ? undefined : true;
   return jsx(Text, {
     as: "time",
-    noWrap: true,
+    noWrap: noWrap,
     ...props
   });
 };
@@ -66012,6 +66057,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 +66102,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 +66204,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 +66408,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();
@@ -76367,6 +76426,12 @@ const PickerFirstResolver = props => {
  *   press, the way a click is the innermost target's. Being a gesture, the open goes through
  *   the interaction gate as one: a `readOnly` picker refuses it and says so
  *   where the finger is, whatever `openWhileReadOnly` says.
+ * @param {number} [maxLines=1] How many lines the value may take before it is
+ *   cut with an ellipsis — `Text`'s `maxLines`, put on the value box, and
+ *   granted to a `BadgeList` in the `ui`, which caps its rows to it (see
+ *   max_lines_context.js). `variant="text"` and `variant="bare"` are not
+ *   clamped: their drawing is the caller's own, a `Text` in the `ui` cuts
+ *   itself with its own `maxLines`. See docs/typography.md.
  * @param {import("ignore:preact").ComponentChildren | "default" | import("ignore:preact").ComponentType} [ui] What the
  *   trigger draws in place of the value's default rendering (a date, a list
  *   joined by commas…) — and then all it draws: `placeholder` is not shown
@@ -76606,6 +76671,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
@@ -78129,13 +78203,7 @@ const renderDayDefault = (day, {
   type: "date",
   format: format,
   dayLabel: true,
-  lang: lang
-  // A Time keeps its date on one line by default; here a day too long for
-  // the box is meant to wrap, so that is undone — except for maxLines={1},
-  // which IS one line and cuts it itself (see Text's own TextOverflow):
-  // saying "you may wrap" there would undo the truncation instead.
-  ,
-  noWrap: maxLines === 1 || maxLines === "1" ? undefined : false,
+  lang: lang,
   maxLines: maxLines,
   children: day
 });