@jsenv/dom 0.17.86 → 0.17.88

package/dist/jsenv_dom.js

@@ -575,21 +575,28 @@ const getEventLabel = (e) => {
 
 const createIterableWeakSet = () => {
   const objectWeakRefSet = new Set();
+  // The one WeakRef of each member, so that adding a member twice keeps one
+  // entry and deleting it once removes it. Weak on the object, and a WeakRef
+  // holds nothing: this keeps nothing alive.
+  let weakRefMap = new WeakMap();
 
   return {
     add: (object) => {
+      if (weakRefMap.has(object)) {
+        return;
+      }
       const objectWeakRef = new WeakRef(object);
       objectWeakRefSet.add(objectWeakRef);
+      weakRefMap.set(object, objectWeakRef);
     },
 
     delete: (object) => {
-      for (const weakRef of objectWeakRefSet) {
-        if (weakRef.deref() === object) {
-          objectWeakRefSet.delete(weakRef);
-          return true;
-        }
+      const objectWeakRef = weakRefMap.get(object);
+      if (!objectWeakRef) {
+        return false;
       }
-      return false;
+      weakRefMap.delete(object);
+      return objectWeakRefSet.delete(objectWeakRef);
     },
 
     *[Symbol.iterator]() {
@@ -604,21 +611,12 @@ const createIterableWeakSet = () => {
     },
 
     has: (object) => {
-      for (const weakRef of objectWeakRefSet) {
-        const objectCandidate = weakRef.deref();
-        if (objectCandidate === undefined) {
-          objectWeakRefSet.delete(weakRef);
-          continue;
-        }
-        if (objectCandidate === object) {
-          return true;
-        }
-      }
-      return false;
+      return weakRefMap.has(object);
     },
 
     clear: () => {
       objectWeakRefSet.clear();
+      weakRefMap = new WeakMap();
     },
 
     get size() {
@@ -9121,7 +9119,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") {
@@ -9413,7 +9411,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 };
@@ -9870,13 +9872,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();

package/package.json

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