@jsenv/navi 0.29.345 → 0.29.347

package/dist/dev/jsenv_navi.js

@@ -50776,9 +50776,15 @@ const COMMAND_DEFAULT_PROPS_FACTORIES = {
  *   drawn — must still close it. What `<Link pressableDuringRouteTransition>`
  *   says, for the other half of a toggle. Only for a control that does not
  *   travel with the pages.
- * @param {Function} [action] On a button with an `href` or a `route`, the
- *   same order as a Link's: it runs on the press, before the navigation, and
- *   the navigation does not wait for it (see Link's `action`).
+ * @param {Function} [action] What the press runs: a button holds no value, so
+ *   every press is one call. The button is busy while it runs, refuses a second
+ *   press and draws the error on itself. On a button with an `href` or a
+ *   `route`, the same order as a Link's: it runs on the press, before the
+ *   navigation, and the navigation does not wait for it (see Link's `action`).
+ * @param {Function} [uiAction] The same press, for what cannot fail and takes
+ *   no time — moving something else on screen, writing a local signal: a plain
+ *   callback, no busy state, nothing drawn if it throws (see
+ *   docs/actions.md#action-or-uiaction).
  * @param {string} [selfInteractions] The interactions this button takes for
  *   itself inside a zone that belongs to another control — a chip's cross on a
  *   picker's façade, an eye on a pressable row, a badge against the edge of a
@@ -68486,6 +68492,7 @@ const useListScrollSync = ({
   };
   useLayoutEffect(resolveScroller);
   useStickyScrollportWarning(ref, scroller);
+  useDuplicateHeaderWarning(ref);
   useStuckWindowWarning({
     ref,
     scrollerElResolved,
@@ -69508,6 +69515,34 @@ const useStickyScrollportWarning = (ref, scroller) => {
     });
   });
 };
+// A list has one header: the row that caps it — the column row of a table —
+// and the box the list measures to keep the others from scrolling under it. A
+// second one takes that same place, so both sit at the capped edge before
+// every row and the rows declared between them read as belonging to the last:
+// a title meant to open a run of rows ends up titling nothing. That title is a
+// group label, which is why this points at List.Group rather than at the
+// stacking.
+const useDuplicateHeaderWarning = ref => {
+  const doneRef = useRef(false);
+  useLayoutEffect(() => {
+    if (doneRef.current) {
+      return;
+    }
+    const listContainerEl = ref.current;
+    if (!listContainerEl) {
+      return;
+    }
+    const headerEls = listContainerEl.querySelectorAll(".navi_list_item_header");
+    if (headerEls.length < 2) {
+      return;
+    }
+    doneRef.current = true;
+    console.warn(`<List> has ${headerEls.length} rows carrying "header", and a list has one: they all stick to the edge it caps, before every row, and the rows declared between them read as belonging to the last one. A title standing over a run of rows is a group: <List.Group label="...">{rows}</List.Group>.`, {
+      list: listContainerEl,
+      headers: [...headerEls]
+    });
+  });
+};
 
 /**
  * "Am I stuck?" — the question a `position: sticky` element cannot ask about
@@ -71902,7 +71937,7 @@ const useItemStore = ({
 };
 
 /**
- * ListGroup — a labeled group of list items.
+ * List.Group — a labeled group of list items.
  *
  * Renders a <li role="presentation"> wrapper containing a label span
  * (accessible via aria-labelledby) and a <ul role="group"> for the items.
@@ -72238,7 +72273,8 @@ const ListResolved = /*#__PURE__*/createComponentResolver([ListFirstResolver, Li
  */
 const List = /*#__PURE__*/Object.assign(ListResolved, {
   Item: ListItem,
-  Items: ListItems
+  Items: ListItems,
+  Group: ListItemGroup
 });
 
 const PickerNaviTime = props => {