Prevent double dragend race condition

clauderic · Feb 8, 2025 · 86ed6c8 · 86ed6c8
1 parent c51778d
commit 86ed6c8
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/.changeset/prevent-double-dragend.md b/.changeset/prevent-double-dragend.md
@@ -0,0 +1,5 @@
+---
+'@dnd-kit/dom': patch
+---
+
+Fixed a regression in the `PointerSensor` where the same drag operation could fire a dragend event twice due to a race condition between `pointerup` and `lostpointercapture`.
diff --git a/packages/dom/src/core/sensors/pointer/PointerSensor.ts b/packages/dom/src/core/sensors/pointer/PointerSensor.ts
@@ -173,7 +173,7 @@ export class PointerSensor extends Sensor<
     ]);
 
     const cleanup = () => {
-      setTimeout(unbindListeners);
+      unbindListeners();
       this.#clearTimeout?.();
       this.initialCoordinates = undefined;
     };
@@ -242,7 +242,7 @@ export class PointerSensor extends Sensor<
     // End the drag and drop operation
     const {status} = this.manager.dragOperation;
 
-    if (!status.idle) {
+    if (status.dragging) {
       // Prevent the default behaviour of the event
       event.preventDefault();
       event.stopPropagation();