@jsenv/navi 0.29.408 → 0.29.409

package/dist/dev/jsenv_navi.js

@@ -13145,12 +13145,13 @@ const useBoundAction = (action, actionParamsSignal) => {
     if (existingNoopAction) {
       return existingNoopAction;
     }
+    // Not bound to the params signal: nothing reads what a noop runs with, and
+    // a proxy following the signal would retarget (a new child action for each
+    // new value) at every change — for every button of a form, at every change
+    // of any of its fields.
     const noopAction = createAction(() => {}, { params: undefined });
-    const noopActionBound = actionParamsSignal
-      ? noopAction.bindParams(actionParamsSignal)
-      : noopAction;
-    noopActionRef.current = noopActionBound;
-    return noopActionBound;
+    noopActionRef.current = noopAction;
+    return noopAction;
   }
   const isFunction = typeof action === "function";
   if (!isFunction) {
@@ -83110,7 +83111,6 @@ const useWheelInteractions = ({
   attemptInteraction,
   cancelAnim,
   setPos,
-  releaseScrub,
   snapPosToRow,
   getItemSize,
   viewportMain,
@@ -83519,8 +83519,6 @@ const useWheelInteractions = ({
       clearTimeout(settleTimer);
       stopClaimingGesture();
       cancelAnim();
-      // Nothing is left to settle the scrub in progress.
-      releaseScrub();
       document.removeEventListener("pointerdown", onDocumentPointerDown, {
         capture: true
       });
@@ -83686,11 +83684,11 @@ function WheelUI(props) {
     ...props,
     ref: inputRef,
     type: "navi_js",
-    // An arrow or a tap changes the value at once while the row glides into
-    // place, and the action must wait for the wheel to come to rest. "custom"
-    // stops setUIState from auto-firing the action on every value change;
-    // commitSelection dispatches it explicitly on settle. A caller can still
-    // override actionEvent.
+    // uiAction fires live (every center-crossing, via setPos live reports);
+    // the committed action must fire only when the wheel stabilizes. "custom"
+    // stops setUIState from auto-firing the action on every value change, so
+    // the live reports are uiAction-only; commitSelection then dispatches the
+    // action explicitly on settle. A caller can still override actionEvent.
     actionEvent: props.actionEvent ?? "custom"
   }, {
     controlType: "input"
@@ -83786,11 +83784,6 @@ function WheelUI(props) {
   // while our own scroll-driven selection does not scroll a second time. Holds the
   // TARGET so rapid inputs step from it and accumulate. null = nothing pending.
   const centeredIndexRef = useRef(null);
-  // A user scrub moved the wheel onto a row that is not the value yet: set at
-  // the first row it crosses, cleared when it settles (commitSelection), when
-  // an arrow or a tap sets a value at once (glideToIndex), or when it is
-  // dropped (releaseScrub). See setPos.
-  const scrubHoldsRowRef = useRef(false);
   // The wheel does NOT use native scroll. `posRef` is our own virtual scroll
   // position (px, main axis); the list track is translated by -pos. This is the
   // single source of truth, wrapped modulo the real-list extent in loop mode, so
@@ -83979,14 +83972,12 @@ function WheelUI(props) {
     posRef.current = clampNumber(posRef.current, 0, (count - 1) * size);
   };
 
-  // `live` marks a user scrub (drag, momentum, wheel scroll). A scrub moves the
-  // drawing, not the value: the row it reaches is held by the wheel, and only
-  // commitSelection turns it into the value, once the motion settles. Reported
-  // at each crossing, every notch would be a value change for all that is
-  // bound to the wheel — its group, its form, a signal, the address, which
-  // records where a value was put down, not the path the finger took
-  // (navigation.md, "A value written per frame"). A programmatic move
-  // (centerOnIndex) omits `live`: it places the value the wheel already has.
+  // `live` (user scrubs: drag, momentum, wheel scroll) reports the value as each
+  // item crosses the center, so uiAction tracks the wheel in real time — like
+  // every other control. The committed action still fires only once it settles
+  // (commitSelection). A programmatic move (centerOnIndex on an external value
+  // sync) omits it, so scrolling the wheel into place doesn't echo the value
+  // back out.
   const setPos = (vp, pos, {
     live = false
   } = {}) => {
@@ -83997,25 +83988,18 @@ function WheelUI(props) {
     renderPos(vp);
     if (live && interactive) {
       const index = centeredIndex(vp);
-      if (index !== centeredIndexRef.current && trackedItemsRef.current[index]) {
-        // In step with the scrub so the next arrow or tap steps from the row
-        // the wheel is on (see currentTargetIndex).
-        centeredIndexRef.current = index;
-        scrubHoldsRowRef.current = true;
+      if (index !== centeredIndexRef.current) {
+        const item = trackedItemsRef.current[index];
+        if (item) {
+          // Keep centeredIndexRef in step so the every-render
+          // syncCenterToSelection doesn't read this as an external change and
+          // fight the in-progress motion.
+          centeredIndexRef.current = index;
+          requestSelectValue(item.value, new CustomEvent("navi_wheel_scrub"));
+        }
       }
     }
   };
-  // The scrub was dropped before it settled (the wheel stopped being
-  // interactive, or its input bindings were torn down): the row it held never
-  // became the value. Forgetting it lets the next sync put the wheel back on
-  // the value.
-  const releaseScrub = () => {
-    if (!scrubHoldsRowRef.current) {
-      return;
-    }
-    scrubHoldsRowRef.current = false;
-    centeredIndexRef.current = null;
-  };
 
   // Returns whether it actually interrupted a motion in flight — lets the caller
   // tell a fresh gesture from one that "caught" a spinning/gliding wheel.
@@ -84051,7 +84035,6 @@ function WheelUI(props) {
       renderPos(vp);
     }
     centeredIndexRef.current = index;
-    scrubHoldsRowRef.current = false;
     if (!interactive || glideFromOutsideRef.current) {
       return;
     }
@@ -84256,7 +84239,6 @@ function WheelUI(props) {
   // inputs accumulate.
   const glideToIndex = (vp, index, event) => {
     centeredIndexRef.current = index;
-    scrubHoldsRowRef.current = false;
     requestSelectValue(trackedItemsRef.current[index].value, event);
     glideTo(vp, glideTargetFor(vp, index));
   };
@@ -84287,12 +84269,12 @@ function WheelUI(props) {
   // Sync the center with the current value — used on first display and whenever
   // the controlled value changes from outside.
   const syncCenterToSelection = (viewportEl, behavior) => {
-    // A fling in flight, or a scrub holding a row, commits on settle: the finger
-    // owns the wheel until then, and the row it leaves the wheel on overrules a
-    // value arriving meanwhile. A glide is different: its target is re-aimed
-    // below (glideTo never restarts the loop), so a value set from outside
-    // mid-glide is where the wheel ends up.
-    if (momentumRef.current !== null || scrubHoldsRowRef.current) {
+    // A fling in flight reports its own value at every row crossing and commits
+    // on settle: the finger that threw it owns the wheel until then, and a value
+    // arriving meanwhile is re-stated by the next crossing anyway. A glide is
+    // different: its target is re-aimed below (glideTo never restarts the
+    // loop), so a value set from outside mid-glide is where the wheel ends up.
+    if (momentumRef.current !== null) {
       return;
     }
     if (trackedItemsRef.current.length === 0) {
@@ -84375,7 +84357,6 @@ function WheelUI(props) {
     attemptInteraction,
     cancelAnim,
     setPos,
-    releaseScrub,
     snapPosToRow,
     getItemSize,
     viewportMain,