Fix #105 Drag-and-drop not working in Chrome

Author: mistic100

bower.json

@@ -21,7 +21,7 @@
     "blanket": "^1.1.0",
     "qunit": "^1.17.0",
     "bootstrap-select": "^1.6.0",
-    "bootbox": "^3.3.0",
+    "bootbox": "^4.3.0",
     "awesome-bootstrap-checkbox": "^0.3.0"
   },
   "keywords": [

src/plugins/sortable/plugin.js

@@ -33,13 +33,15 @@ QueryBuilder.define('sortable', function(options) {
 
             src = Model(e.target);
 
-            var ph = $('<div class="rule-placeholder">&nbsp;</div>');
-            ph.css('min-height', src.$el.height());
+            // Chrome glitchs
+            // - helper invisible if hidden immediately
+            // - "dragend" is called immediately if we modify the DOM directly
+            setTimeout(function() {
+                var ph = $('<div class="rule-placeholder">&nbsp;</div>');
+                ph.css('min-height', src.$el.height());
 
-            placeholder = src.parent.addRule(ph, src.getPos());
+                placeholder = src.parent.addRule(ph, src.getPos());
 
-            // Chrome glitch (helper invisible if hidden immediately)
-            setTimeout(function() {
                 src.$el.hide();
             }, 0);
         });
@@ -49,7 +51,9 @@ QueryBuilder.define('sortable', function(options) {
             e.preventDefault();
             e.stopPropagation();
 
-            moveSortableToTarget(placeholder, $(e.target));
+            if (placeholder) {
+                moveSortableToTarget(placeholder, $(e.target));
+            }
         });
 
         // dragover: prevent glitches

tests/data.module.js

@@ -58,11 +58,6 @@ $(function(){
       /operator_not_multiple/
     );
 
-    assert.validationError($b,
-      { id: 'select' },
-      /select_empty/
-    );
-
     assert.validationError($b,
       { id: 'select_mult' },
       /select_empty/