removeOnSpill working on mobile closes bevacqua#98

Author: bevacqua

changelog.markdown

@@ -3,6 +3,7 @@
 - Fixed a _historical_ bug, where click on anchors would be ignored within `dragula` containers in mobile
 - Fixed a bug where events wouldn't be gracefully removed if `drake` were destroyed during a drag event
 - Now emits `dragend` after `out` to preserve consistency _(because `drag` is emitted before `over`)_
+- Fixed another old bug where attempting to remove elements using `removeOnSpill` on mobile would fail
 
 # 3.0.1 Carjacking
 

dragula.js

@@ -46,6 +46,10 @@ function dragula (initialContainers, options) {
     dragging: false
   });
 
+  if (o.removeOnSpill === true) {
+    drake.on('over', spillOver).on('out', spillOut);
+  }
+
   events();
 
   return drake;
@@ -338,7 +342,7 @@ function dragula (initialContainers, options) {
       reference = _initialSibling;
       dropTarget = _source;
     } else {
-      if ((o.copy || o.removeOnSpill === true) && item.parentElement) {
+      if (o.copy && item.parentElement) {
         item.parentElement.removeChild(item);
       }
       return;
@@ -358,6 +362,14 @@ function dragula (initialContainers, options) {
     function out () { if (_lastDropTarget) { moved('out'); } }
   }
 
+  function spillOver (el) {
+    classes.rm(el, 'gu-hide');
+  }
+
+  function spillOut (el) {
+    if (drake.dragging) { classes.add(el, 'gu-hide'); }
+  }
+
   function renderMirrorImage () {
     if (_mirror) {
       return;