@jsenv/dom 0.17.85 → 0.17.87

package/dist/jsenv_dom.js

@@ -4976,14 +4976,19 @@ const getFocusVisibilityInfo = (node, { excludeAriaHidden } = {}) => {
       }
       // Continue checking ancestors
     }
-    if (elementIsDialog(nodeOrAncestor) && !nodeOrAncestor.open) {
-      return { visible: false, reason: "inside closed dialog element" };
-    }
+    const isPopover =
+      nodeOrAncestor.popover !== null && nodeOrAncestor.popover !== undefined;
+    const isOpenPopover = isPopover && nodeOrAncestor.matches(":popover-open");
+    // A dialog shown as a popover (showPopover()) is open without `open`,
+    // which only show() and showModal() set.
     if (
-      nodeOrAncestor.popover !== null &&
-      nodeOrAncestor.popover !== undefined &&
-      !nodeOrAncestor.matches(":popover-open")
+      elementIsDialog(nodeOrAncestor) &&
+      !nodeOrAncestor.open &&
+      !isOpenPopover
     ) {
+      return { visible: false, reason: "inside closed dialog element" };
+    }
+    if (isPopover && !isOpenPopover) {
       return { visible: false, reason: "inside closed popover element" };
     }
     // Open popovers and open dialogs render in the top layer: they escape
@@ -4992,11 +4997,7 @@ const getFocusVisibilityInfo = (node, { excludeAriaHidden } = {}) => {
     if (elementIsDialog(nodeOrAncestor) && nodeOrAncestor.open) {
       break;
     }
-    if (
-      nodeOrAncestor.popover !== null &&
-      nodeOrAncestor.popover !== undefined &&
-      nodeOrAncestor.matches(":popover-open")
-    ) {
+    if (isOpenPopover) {
       break;
     }
     nodeOrAncestor = nodeOrAncestor.parentNode;
@@ -9120,7 +9121,7 @@ const createDragGestureController = (options = {}) => {
     // Where the pointer IS is not where it is going: throwing something is a
     // matter of speed, and the gesture is the only place that sees the timing of
     // the events it receives.
-    const measureVelocity = createVelocityMeter(grabX, grabY);
+    const measureVelocity = createVelocityMeter(grabX, grabY, event.timeStamp);
 
     document_interactions: {
       if (documentInteractions === "manual") {
@@ -9412,7 +9413,11 @@ const createDragGestureController = (options = {}) => {
         dragEvent: event,
         isRelease,
       });
-      const [velocityX, velocityY] = measureVelocity(dragX, dragY);
+      const [velocityX, velocityY] = measureVelocity(
+        dragX,
+        dragY,
+        event.timeStamp,
+      );
       const startedPrevious = gestureInfo.started;
       const layoutPrevious = gestureInfo.layout;
       // previousGestureInfo = { ...gestureInfo };
@@ -9869,13 +9874,18 @@ const isPrimaryButtonEvent = (event) =>
  * A pointer held still keeps producing samples at the same place, so the window
  * empties itself of movement and the speed falls back to zero on its own: put
  * down slowly is not thrown.
+ * Each sample carries the time of its event (`timeStamp`, the clock of
+ * `performance.now()`), not the time it is handled: behind a long frame, every
+ * report the hand made meanwhile is handled in one burst with the release, and
+ * stamped then, the last few pixels — often a thumb drifting back as it lifts —
+ * fill the window and read as a throw the other way. A programmatic event is
+ * stamped at its creation, which is when it happens.
  */
 const VELOCITY_WINDOW_MS = 100;
-const createVelocityMeter = (grabX, grabY) => {
-  const samples = [{ time: performance.now(), x: grabX, y: grabY }];
+const createVelocityMeter = (grabX, grabY, grabTime) => {
+  const samples = [{ time: grabTime, x: grabX, y: grabY }];
 
-  const measureVelocity = (x, y) => {
-    const time = performance.now();
+  const measureVelocity = (x, y, time) => {
     samples.push({ time, x, y });
     while (samples.length > 2 && time - samples[1].time > VELOCITY_WINDOW_MS) {
       samples.shift();
@@ -15039,10 +15049,11 @@ const startDragTo = (
  * A press that WOULD be a drag, and is not.
  *
  * Recognized exactly as `startDragTo` recognizes it: the intent is established by
- * the same threshold — a mouse travelling, a finger holding still, the first
- * pixel inside a `[data-drag-on-contact]`. What differs is what happens once it
- * is established: nothing is grabbed, nothing translates, and `onRefuse` is told
- * at the instant the grab would have been acquired.
+ * the same threshold — a mouse travelling a few pixels, a finger holding still,
+ * or travelling those same few pixels inside a `[data-drag-on-contact]`. What
+ * differs is what happens once it is established: nothing is grabbed, nothing
+ * translates, and `onRefuse` is told at the instant the grab would have been
+ * acquired.
  *
  * That instant is the whole point. An object that stays put under the hand and
  * says nothing reads as a screen that is broken, and the hand pulls harder; the

package/package.json

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