@jsenv/dom 0.17.22 → 0.17.23

package/dist/jsenv_dom.js

@@ -4295,41 +4295,77 @@ const createPreviousNodeIterator = (fromNode, rootNode, skipRoot = null) => {
  * link or a button that click means "follow me", which is not what the hand
  * asked for: the press was already answered, by the gesture.
  *
- * So it is swallowed, once, in capture on the document — before any handler an
- * element may have, and without anyone having to know which element that is.
+ * So it is swallowed, once, before any other listener sees it. Being first is
+ * earned twice, because both orderings matter:
+ *
+ * - on `window`, the first target of the capture phase — a listener anywhere
+ *   lower (document included) comes after, no matter when it was registered.
+ * - registered at module load — among listeners on the same target and phase,
+ *   registration order decides. A listener added when the gesture ends would
+ *   lose to any window-capture listener registered at startup (@jsenv/navi's
+ *   link interception is one), so the listener is permanent and merely armed
+ *   by each gesture.
+ *
+ * A window-capture listener that this module's evaluation cannot be proven to
+ * precede must not bet on that order: it checks `clickIsSuppressed()` and
+ * stands aside on its own.
  */
 
+let suppressing = false;
+let disarmAtNextPress = false;
+
+const suppressClick = (clickEvent) => {
+  if (!suppressing) {
+    return;
+  }
+  suppressing = false;
+  disarmAtNextPress = false;
+  clickEvent.preventDefault();
+  clickEvent.stopImmediatePropagation();
+};
+const onPointerDown = () => {
+  if (disarmAtNextPress) {
+    suppressing = false;
+    disarmAtNextPress = false;
+  }
+};
+window.addEventListener("click", suppressClick, { capture: true });
+window.addEventListener("pointerdown", onPointerDown, { capture: true });
+
 /**
  * Swallows the next click, for a gesture that has just answered the press.
  *
- * @returns {() => void} the gesture is over. The suppressor cannot be taken
- *   down with it — the click is dispatched AFTER the pointerup that ends the
+ * @returns {() => void} the gesture is over. The suppression cannot be lifted
+ *   with it — the click is dispatched AFTER the pointerup that ends the
  *   gesture, so it would be gone one event too early, and the drag would end on
- *   the link it started from being followed. It goes once it has swallowed a
+ *   the link it started from being followed. It lifts once it has swallowed a
  *   click, or at the next press if the gesture produced none: a click is always
- *   preceded by a press, so a suppressor that outlives one press can never
+ *   preceded by a press, so a suppression that outlives one press can never
  *   reach the click of another.
  */
 const suppressClickAfterGesture = () => {
-  const suppressClick = (clickEvent) => {
-    clickEvent.stopPropagation();
-    clickEvent.preventDefault();
-    stopSuppressing();
-  };
-  const stopSuppressing = () => {
-    document.removeEventListener("click", suppressClick, { capture: true });
-    document.removeEventListener("pointerdown", stopSuppressing, {
-      capture: true,
-    });
-  };
-  document.addEventListener("click", suppressClick, { capture: true });
+  suppressing = true;
+  disarmAtNextPress = false;
   return () => {
-    document.addEventListener("pointerdown", stopSuppressing, {
-      capture: true,
-    });
+    disarmAtNextPress = true;
   };
 };
 
+/**
+ * Whether the click being dispatched is one a gesture left behind — armed by
+ * `suppressClickAfterGesture`, waiting to be swallowed by this module.
+ *
+ * A last resort, not a convenience. The suppressor already swallows the click
+ * before anyone else sees it; the one listener that legitimately needs to ask
+ * is a `click` listener in capture on `window` whose registration cannot be
+ * proven to come after this module's evaluation — that one may run before the
+ * suppressor and must stand aside on its own. Everywhere else (an element,
+ * `document`, the bubble phase) the click never arrives and checking this is
+ * dead code. Reach for it only when you are sure that is your situation and
+ * no other ordering is available.
+ */
+const clickIsSuppressed = () => suppressing;
+
 /**
  * A press that says something by NOT moving.
  *
@@ -19623,4 +19659,4 @@ const useResizeStatus = (elementRef, { as = "number" } = {}) => {
   };
 };
 
-export { EASING, ELEMENT_SIZE_CHANGE, activeElementSignal, addActiveElementEffect, addAttributeEffect, allowWheelThrough, appendStyles, applyNewPosition, canScroll, captureScrollState, chainEvent, claimWheelGesture, closestOpenableAncestor, contrastColor, createBackgroundColorTransition, createBackgroundTransition, createBorderRadiusTransition, createBorderTransition, createDragGestureController, createDragToMoveGestureController, createEventGroupLogger, createGroupTransitionController, createHeightTransition, createIterableWeakSet, createOpacityTransition, createPubSub, createStyleController, createTimelineTransition, createTransition, createTranslateXTransition, createValueEffect, createWidthTransition, cubicBezier, dispatchCustomEvent, dispatchInternalCustomEvent, dispatchPublicCustomEvent, dragAfterIntent, elementIsFocusable, elementIsVisibleForFocus, elementIsVisuallyVisible, findAfter, findAncestor, findBefore, findDescendant, findEvent, findFocusDelegateTarget, findFocusable, findSelfOrAncestorFixedPosition, formatEventSideEffect, getAncestorOpenType, getAvailableHeight, getAvailableWidth, getBackground, getBackgroundColor, getBorder, getBorderRadius, getBorderSizes, getContrastRatio, getDefaultStyles, getDragCoordinates, getDropTargetInfo, getElementSignature, getFirstVisuallyVisibleAncestor, getFocusVisibilityInfo, getHeight, getHeightWithoutTransition, getInnerHeight, getInnerWidth, getKeyboardEventDefaultAction, getLuminance, getMarginSizes, getMaxHeight, getMaxWidth, getMinHeight, getMinWidth, getOpacity, getOpacityWithoutTransition, getPaddingSizes, getPositionedParent, getPositioningScrollOffset, getPreferedColorScheme, getScrollBox, getScrollContainer, getScrollContainerSet, getScrollRelativeRect, getSelfAndAncestorScrolls, getStyle, getTranslateX, getTranslateXWithoutTransition, getTranslateY, getVirtualKeyboardOverlayHeight, getVisuallyVisibleInfo, getWidth, getWidthWithoutTransition, hasCSSSizeUnit, initFlexDetailsSet, initFocusGroup, initPositionSticky, isAncestorOpen, isPrimaryButtonEvent, isSameColor, isScrollable, markDragSource, measureLongestVisualLineWidth, measureScrollbar, measureWidestChildRow, mergeOneStyle, mergeTwoStyles, normalizeKeyboardKey, normalizeStyle, normalizeStyles, observeAncestorOpenState, onAncestorReopen, parsePositionArea, parseStyle, performTabNavigation, pickPositionRelativeTo, prefersDarkColors, prefersLightColors, preventFocusNav, preventFocusNavViaKeyboard, preventIntermediateScrollbar, releaseWheelGesture, resolveCSSColor, resolveCSSSize, resolveColorLuminance, resolveOklchLightness, scrollIntoViewScoped, scrollIntoViewWithStickyAwareness, scrollRoomTowards, setAttribute, setAttributes, setStyles, setVirtualKeyboardOverlaysContent, snapToPixel, startDragTo, startDragToResizeGesture, startDragToTravel, stickyAsRelativeCoords, stringifyStyle, subscribeVirtualKeyboardGeometryChange, subscribeVisualViewportResizeSettled, subscribeWindowResizeSettled, suppressClickAfterGesture, trapFocusInside, trapScrollInside, useActiveElement, useAvailableHeight, useAvailableWidth, useMaxHeight, useMaxWidth, useResizeStatus, visibleRectEffect, waitForPressHeld, watchWheelTravel, wheelGestureIsTakenFrom };
+export { EASING, ELEMENT_SIZE_CHANGE, activeElementSignal, addActiveElementEffect, addAttributeEffect, allowWheelThrough, appendStyles, applyNewPosition, canScroll, captureScrollState, chainEvent, claimWheelGesture, clickIsSuppressed, closestOpenableAncestor, contrastColor, createBackgroundColorTransition, createBackgroundTransition, createBorderRadiusTransition, createBorderTransition, createDragGestureController, createDragToMoveGestureController, createEventGroupLogger, createGroupTransitionController, createHeightTransition, createIterableWeakSet, createOpacityTransition, createPubSub, createStyleController, createTimelineTransition, createTransition, createTranslateXTransition, createValueEffect, createWidthTransition, cubicBezier, dispatchCustomEvent, dispatchInternalCustomEvent, dispatchPublicCustomEvent, dragAfterIntent, elementIsFocusable, elementIsVisibleForFocus, elementIsVisuallyVisible, findAfter, findAncestor, findBefore, findDescendant, findEvent, findFocusDelegateTarget, findFocusable, findSelfOrAncestorFixedPosition, formatEventSideEffect, getAncestorOpenType, getAvailableHeight, getAvailableWidth, getBackground, getBackgroundColor, getBorder, getBorderRadius, getBorderSizes, getContrastRatio, getDefaultStyles, getDragCoordinates, getDropTargetInfo, getElementSignature, getFirstVisuallyVisibleAncestor, getFocusVisibilityInfo, getHeight, getHeightWithoutTransition, getInnerHeight, getInnerWidth, getKeyboardEventDefaultAction, getLuminance, getMarginSizes, getMaxHeight, getMaxWidth, getMinHeight, getMinWidth, getOpacity, getOpacityWithoutTransition, getPaddingSizes, getPositionedParent, getPositioningScrollOffset, getPreferedColorScheme, getScrollBox, getScrollContainer, getScrollContainerSet, getScrollRelativeRect, getSelfAndAncestorScrolls, getStyle, getTranslateX, getTranslateXWithoutTransition, getTranslateY, getVirtualKeyboardOverlayHeight, getVisuallyVisibleInfo, getWidth, getWidthWithoutTransition, hasCSSSizeUnit, initFlexDetailsSet, initFocusGroup, initPositionSticky, isAncestorOpen, isPrimaryButtonEvent, isSameColor, isScrollable, markDragSource, measureLongestVisualLineWidth, measureScrollbar, measureWidestChildRow, mergeOneStyle, mergeTwoStyles, normalizeKeyboardKey, normalizeStyle, normalizeStyles, observeAncestorOpenState, onAncestorReopen, parsePositionArea, parseStyle, performTabNavigation, pickPositionRelativeTo, prefersDarkColors, prefersLightColors, preventFocusNav, preventFocusNavViaKeyboard, preventIntermediateScrollbar, releaseWheelGesture, resolveCSSColor, resolveCSSSize, resolveColorLuminance, resolveOklchLightness, scrollIntoViewScoped, scrollIntoViewWithStickyAwareness, scrollRoomTowards, setAttribute, setAttributes, setStyles, setVirtualKeyboardOverlaysContent, snapToPixel, startDragTo, startDragToResizeGesture, startDragToTravel, stickyAsRelativeCoords, stringifyStyle, subscribeVirtualKeyboardGeometryChange, subscribeVisualViewportResizeSettled, subscribeWindowResizeSettled, suppressClickAfterGesture, trapFocusInside, trapScrollInside, useActiveElement, useAvailableHeight, useAvailableWidth, useMaxHeight, useMaxWidth, useResizeStatus, visibleRectEffect, waitForPressHeld, watchWheelTravel, wheelGestureIsTakenFrom };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/dom",
-  "version": "0.17.22",
+  "version": "0.17.23",
   "type": "module",
   "description": "DOM utilities for writing frontend code",
   "repository": {