@jsenv/navi 0.29.320 → 0.29.322

package/dist/jsenv_navi.js

@@ -4,7 +4,7 @@
  */
 import { installImportMetaCssBuild, windowHeightSignal, windowWidthSignal, visualViewportHeightSignal, visualViewportWidthSignal, getAppHeight, getAppWidth, coarsePointerSignal, smallTouchScreenSignal } from "./jsenv_navi_side_effects.js";
 export { disableVirtualKeyboardOverlay } from "./jsenv_navi_side_effects.js";
-import { elementIsFocusable, createIterableWeakSet, dispatchInternalCustomEvent, dispatchCustomEvent, getVisuallyVisibleInfo, getFirstVisuallyVisibleAncestor, getElementSignature, createPubSub, findEvent, createValueEffect, findFocusDelegateTarget, findFocusable, scrollIntoViewThroughScrollables, allowWheelThrough, dispatchPublicCustomEvent, resolveCSSColor, ELEMENT_SIZE_CHANGE, findSelfOrAncestorFixedPosition, visibleRectEffect, pickPositionRelativeTo, getBorderSizes, getPaddingSizes, applyNewPosition, measureLongestVisualLineWidth, chainEvent, keepTouchRefusable, isPressDrivenClick, waitForTap, waitForPressHeld, suppressClickAfterGesture, startDragToTravel, markDragSource, refuseDragTo, startDragTo, installPanZoom, getKeyboardEventDefaultAction, activeElementSignal, normalizeStyle, mergeOneStyle, getPositionedParent, normalizeStyles, createGroupTransitionController, getBorderRadius, preventIntermediateScrollbar, createOpacityTransition, watchWheelTravel, scrollRoomTowards, getScrollContainer, isTouchDrivenEvent, scrollIntoViewScoped, closestOpenableAncestor, isAncestorOpen, isDisplayedDespiteClosedAncestor, observeAncestorOpenState, getAncestorOpenType, findBefore, findAfter, resolveCSSSize, hasCSSSizeUnit, releaseWheelGesture, getScrollIntoViewScopedOffsets, wheelGestureIsTakenFrom, claimWheelGesture, initFocusGroup, stringifyStyle as stringifyStyle$1, resolveOklchLightness, contrastColor, parsePositionArea, snapToPixel, trapFocusInside, trapScrollInside, onAncestorReopen, isPressDisputedByDrag, canScroll, measureWidestChildRow, performTabNavigation, dragAfterIntent, stickyAsRelativeCoords, createDragToMoveGestureController, getDropTargetInfo, setStyles, useActiveElement } from "@jsenv/dom";
+import { elementIsFocusable, createIterableWeakSet, dispatchInternalCustomEvent, dispatchCustomEvent, getVisuallyVisibleInfo, getFirstVisuallyVisibleAncestor, getElementSignature, createPubSub, findEvent, createValueEffect, findFocusDelegateTarget, findFocusable, scrollIntoViewThroughScrollables, allowWheelThrough, dispatchPublicCustomEvent, resolveCSSColor, ELEMENT_SIZE_CHANGE, findSelfOrAncestorFixedPosition, visibleRectEffect, pickPositionRelativeTo, getBorderSizes, getPaddingSizes, applyNewPosition, measureLongestVisualLineWidth, chainEvent, keepTouchRefusable, isPressDrivenClick, waitForTap, waitForPressHeld, suppressClickAfterGesture, startDragToTravel, dragSourceThatStoodDown, markDragSource, refuseDragTo, startDragTo, installPanZoom, getKeyboardEventDefaultAction, activeElementSignal, normalizeStyle, mergeOneStyle, getPositionedParent, normalizeStyles, createGroupTransitionController, getBorderRadius, preventIntermediateScrollbar, createOpacityTransition, watchWheelTravel, scrollRoomTowards, getScrollContainer, isTouchDrivenEvent, scrollIntoViewScoped, closestOpenableAncestor, isAncestorOpen, isDisplayedDespiteClosedAncestor, observeAncestorOpenState, getAncestorOpenType, findBefore, findAfter, resolveCSSSize, hasCSSSizeUnit, releaseWheelGesture, getScrollIntoViewScopedOffsets, wheelGestureIsTakenFrom, claimWheelGesture, initFocusGroup, stringifyStyle as stringifyStyle$1, resolveOklchLightness, contrastColor, parsePositionArea, snapToPixel, trapFocusInside, trapScrollInside, onAncestorReopen, isPressDisputedByDrag, canScroll, measureWidestChildRow, performTabNavigation, dragAfterIntent, stickyAsRelativeCoords, createDragToMoveGestureController, getDropTargetInfo, setStyles, useActiveElement } from "@jsenv/dom";
 export { chainEvent, clickIsSuppressed, contrastColor, createDragGestureController, dragAfterIntent, findEvent, markDragSource, startDragTo } from "@jsenv/dom";
 import { signal, computed, effect, untracked, batch, useComputed, useSignal } from "@preact/signals";
 import { isValidElement, createContext, render, h, toChildArray, options, cloneElement, Fragment as Fragment$1 } from "preact";
@@ -10544,7 +10544,17 @@ defineInteractionDetector({
           },
         });
       }
-      if (hasLongPress) {
+      // A press on something INSIDE this element that is picked up and
+      // carried is that thing's, the same nearer word as the inner hold below:
+      // a finger held still on a piece is the piece's own hold when it drags
+      // by holding, and when it drags by distance it has decided nothing yet
+      // at the delay — a hold answered there is a hold on a piece, and a carry
+      // out of the same press the moment the hand moves. A surface that pans
+      // steps back from the same source at the press (see pan_zoom.js).
+      if (
+        hasLongPress &&
+        !isPressOnDragSourceInside(pointerDownEvent, element)
+      ) {
         // A hold declared INSIDE this element, on this same press: the nearer
         // one answers, the way a click is the innermost target's, and this
         // wait is given up — two sheets opening from one hold is nobody's
@@ -10711,6 +10721,23 @@ const startSwipe = (
   });
 };
 
+// Whether the press landed on a drag source strictly inside the element — the
+// element itself being one is the other arrangement, where the grab answers
+// the hold (see refuseDragTo in @jsenv/dom). A source that stood down from this
+// press carries nothing, so it is no reason to hold back, and the walk goes on
+// above it the way a surface's does.
+const isPressOnDragSourceInside = (pointerDownEvent, element) => {
+  const target = pointerDownEvent.target;
+  if (!target || typeof target.closest !== "function") {
+    return false;
+  }
+  let source = target.closest("[data-drag-source]");
+  if (source && source === dragSourceThatStoodDown(pointerDownEvent)) {
+    source = source.parentElement?.closest("[data-drag-source]") || null;
+  }
+  return Boolean(source) && source !== element && element.contains(source);
+};
+
 // Whether a press landing now completes the tap before it: the same hand, soon
 // enough, and near enough. Axis by axis rather than as a distance, the way every
 // other slop in this family is read.
@@ -34956,6 +34983,8 @@ const createDisplayedEvent = (ancestor, becauseAncestorOpened) => {
  * @returns {Function} triggerAutofocus — can be called manually with a synthetic
  *   event to re-run the focus logic outside of the layout-effect lifecycle.
  */
+const NO_ELEMENT_REF = { current: null };
+
 const useAutoFocus = (
   focusableElementRef,
   autoFocus,
@@ -35087,8 +35116,15 @@ const useAutoFocus = (
     };
   };
 
+  // Nothing to place when the control asks for no focus ("restore" is given
+  // focus by a transfer, never by this effect): the displayed-effect is then
+  // handed no element at all, so it never asks the layout whether one is on
+  // screen — a question every control in a list would otherwise ask at
+  // mount, for an answer this hook would not use. Read at mount, like the
+  // deps below say.
+  const claimsFocus = Boolean(autoFocus) && autoFocus !== "restore";
   useDisplayedLayoutEffect(
-    focusableElementRef,
+    claimsFocus ? focusableElementRef : NO_ELEMENT_REF,
     (el, e) => {
       return triggerAutofocus(e);
     },