@jsenv/navi 0.28.7 → 0.28.9

package/dist/jsenv_navi.js

@@ -5353,10 +5353,12 @@ const stateSignal = (defaultValue, options = {}) => {
   // Convert numeric IDs to strings for consistency
   const signalIdString = String(signalId);
   if (globalSignalRegistry.has(signalIdString)) {
-    const conflictInfo = globalSignalRegistry.get(signalIdString);
-    throw new Error(
-      `Signal ID conflict: A signal with ID "${signalIdString}" already exists (existing default: ${conflictInfo.options.getDefaultValue()})`,
-    );
+    const existingEntry = globalSignalRegistry.get(signalIdString);
+    {
+      throw new Error(
+        `Signal ID conflict: A signal with ID "${signalIdString}" already exists (existing default: ${existingEntry.options.getDefaultValue()})`,
+      );
+    }
   }
 
   // Determine localStorage key: use id if persists=true, or legacy localStorage option
@@ -5481,7 +5483,12 @@ const stateSignal = (defaultValue, options = {}) => {
     // can reflect the current input state without silently correcting it.
     return validity.value;
   };
-  const preactSignal = signal(processValue(getFallbackValue()));
+  const preactSignal = signal(
+    processValue(
+      getFallbackValue()
+        ,
+    ),
+  );
 
   // Override the value setter on the instance to intercept writes and apply processValue.
   // We do this on the instance (not the prototype) so preactSignal remains a real Signal
@@ -23323,6 +23330,7 @@ const CONTROL_ATTRIBUTE_SET = new Set([
   "command",
   "commandFor",
   "command-value", // not standard but make sense, allow to give param to the command in question
+  "list",
 
   // "ui-action-target",
   "navi-input-type",
@@ -31229,6 +31237,11 @@ const generateCrossPlatformCombination = (combination) => {
   return crossPlatform;
 };
 const keyboardEventIsMatchingKeyCombination = (event, keyCombination) => {
+  if (!event.key) {
+    // Some keydown events carry no key at all — the browser synthesizes one
+    // when a native autofill/search suggestion is clicked. No key, no match.
+    return false;
+  }
   const keys = keyCombination.toLowerCase().split("+");
   const activeModifiers = new Set();
   for (const eventProperty of Object.keys(modifierKeyMapping)) {
@@ -41485,7 +41498,10 @@ const css$y = /* css */`
  * @param {boolean|"last-resort"|"restore"} [props.autoFocus="last-resort"] - See
  *   `focus_transfer.js` — `"last-resort"` focuses the popover itself only if it
  *   has no other focusable descendant, `"restore"` keeps it out of the
- *   opening focus chain unless it held focus when the popover closed.
+ *   opening focus chain unless it held focus when the popover closed. `false`
+ *   disables the open-time focus transfer entirely: nothing inside the popover
+ *   receives focus, whoever had the keyboard keeps it — the combobox case,
+ *   where suggestions open under an input being typed in.
  * @param {boolean} [props.open] - Controlled open state.
  * @param {boolean} [props.defaultOpen] - Uncontrolled, mount-only initial
  *   open state — plays no entrance animation (nothing was ever shown as
@@ -42190,7 +42206,12 @@ const usePopoverProps = props => {
       }
     }
     const cancelOpenInteractionSuppression = !silent && hasCssTransitionAnimation ? suppressPointerEventsDuringTransition(popoverEl) : null;
-    const restoreFocus = openController.transferFocusOnOpen(popoverEl);
+    // autoFocus={false}: opening moves no focus at all — whoever has the
+    // keyboard keeps it (combobox-style suggestions under an input being
+    // typed in). The element-level useAutoFocus hook is already a no-op for
+    // false; this skips the open transfer too, and with no transfer there is
+    // nothing to restore on close.
+    const restoreFocus = autoFocus === false ? null : openController.transferFocusOnOpen(popoverEl);
     debugPopup(e, isTopLayer ? `openPopover() -> anchor: ${anchorElement?.tagName}, hasAnchorElement: ${hasAnchorElement}` : `openPopover() -> scroll-container (local)`);
 
     // The close callback openEffect returns — also inlined for the same
@@ -42228,7 +42249,7 @@ const usePopoverProps = props => {
           }
         });
       }
-      restoreFocus(closeEvent);
+      restoreFocus?.(closeEvent);
 
       // pickPositionRelativeTo (visible_rect.js) writes data-position-y/x-
       // current unconditionally on every call, and treats their mere
@@ -48288,6 +48309,10 @@ const css$o = /* css */`
   .navi_picker_spin {
     /* What the loading outline is drawn around. */
     position: relative;
+    /* Written in the control font, like the picker it wraps: the day and its
+       two chevrons are a control, not running text. */
+    font-size: var(--navi-control-font-size);
+    font-family: var(--navi-control-font-family);
     /* Framed like every other control (see navi_css_vars.js): what one steps
        through is a value one edits, and a box around it is what says so. Said
        as CSS rather than as defaults on the Box, so borderWidth="0" or a radius
@@ -51001,7 +51026,11 @@ const useWheelInteractions = ({
       if (vp.contains(documentWheelEvent.target)) {
         return; // on the scroll surface → its own onWheel handles it
       }
-      documentWheelEvent.preventDefault();
+      if (documentWheelEvent.cancelable) {
+        // cancelable=false during an inertial scroll the browser already owns;
+        // preventDefault would be ignored with an intervention warning.
+        documentWheelEvent.preventDefault();
+      }
       keepClaimingGesture(); // the gesture continues elsewhere → keep swallowing it
     };
     const stopClaimingGesture = () => {
@@ -51301,7 +51330,11 @@ const useWheelInteractions = ({
     // preventing touchstart has wider side effects. Requires a non-passive
     // listener. See docs/MOBILE_TAP_SUPPRESSION_AFTER_DRAG.md.
     const onTouchMove = e => {
-      if (drag) {
+      // cancelable=false when the touch landed while a scroll was already in
+      // progress (e.g. the page still coasting from a fling): the browser owns
+      // that scroll, preventDefault would be a no-op and Chrome logs an
+      // intervention warning for the attempt.
+      if (drag && e.cancelable) {
         e.preventDefault();
       }
     };