svelte-dnd-action 0.9.69 → 0.9.70

package/dist/index.js

@@ -1953,6 +1953,11 @@
       originDropZoneRoot.appendChild(draggedEl);
       // We will keep the original dom node in the dom because touch events keep firing on it, we want to re-add it after the framework removes it
       function keepOriginalElementInDom() {
+        // if the drag was already finalized (cleanupPostDrop cleared draggedEl), this rAF loop
+        // must not re-arm the watcher: the original element might leave the DOM long after the
+        // drop (ex: its component unmounts), which would call watchDraggedElement() with an
+        // undefined draggedEl and crash inside observe() (getBoundingClientRect of undefined)
+        if (!draggedEl) return;
         if (!originalDragTarget.parentElement) {
           originalDragTarget.setAttribute(ORIGINAL_DRAGGED_ITEM_MARKER_ATTRIBUTE, true);
           originDropZoneRoot.appendChild(originalDragTarget);

package/dist/index.mjs

@@ -1947,6 +1947,11 @@ function dndzone$2(node, options) {
     originDropZoneRoot.appendChild(draggedEl);
     // We will keep the original dom node in the dom because touch events keep firing on it, we want to re-add it after the framework removes it
     function keepOriginalElementInDom() {
+      // if the drag was already finalized (cleanupPostDrop cleared draggedEl), this rAF loop
+      // must not re-arm the watcher: the original element might leave the DOM long after the
+      // drop (ex: its component unmounts), which would call watchDraggedElement() with an
+      // undefined draggedEl and crash inside observe() (getBoundingClientRect of undefined)
+      if (!draggedEl) return;
       if (!originalDragTarget.parentElement) {
         originalDragTarget.setAttribute(ORIGINAL_DRAGGED_ITEM_MARKER_ATTRIBUTE, true);
         originDropZoneRoot.appendChild(originalDragTarget);

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "svelte-dnd-action",
     "description": "*An awesome drag and drop library for Svelte 3 and 4 (not using the browser's built-in dnd, thanks god): Rich animations, nested containers, touch support and more *",
-    "version": "0.9.69",
+    "version": "0.9.70",
     "repository": {
         "type": "git",
         "url": "git+https://github.com/isaacHagoel/svelte-dnd-action.git"

package/src/pointerAction.js

@@ -511,6 +511,11 @@ export function dndzone(node, options) {
         originDropZoneRoot.appendChild(draggedEl);
         // We will keep the original dom node in the dom because touch events keep firing on it, we want to re-add it after the framework removes it
         function keepOriginalElementInDom() {
+            // if the drag was already finalized (cleanupPostDrop cleared draggedEl), this rAF loop
+            // must not re-arm the watcher: the original element might leave the DOM long after the
+            // drop (ex: its component unmounts), which would call watchDraggedElement() with an
+            // undefined draggedEl and crash inside observe() (getBoundingClientRect of undefined)
+            if (!draggedEl) return;
             if (!originalDragTarget.parentElement) {
                 originalDragTarget.setAttribute(ORIGINAL_DRAGGED_ITEM_MARKER_ATTRIBUTE, true);
                 originDropZoneRoot.appendChild(originalDragTarget);