Fix one bug and another pops up. This fixes issue where passing detachNode=false to removeWidget would still cause it to be removed from the DOM.

Author: radiolips

dist/gridstack.js

@@ -335,12 +335,14 @@
         return node;
     };
 
-    GridStackEngine.prototype.removeNode = function(node) {
+    GridStackEngine.prototype.removeNode = function(node, detachNode) {
         this._removedNodes.push(_.clone(node));
         node._id = null;
         this.nodes = _.without(this.nodes, node);
         this._packNodes();
-        this._notify(node);
+        if (typeof detachNode != 'undefined' && detachNode) {
+            this._notify(node);
+        }
     };
 
     GridStackEngine.prototype.canMoveNode = function(node, x, y, width, height) {
@@ -1083,7 +1085,7 @@
             node = this.grid.getNodeDataByDOMEl(el);
         }
 
-        this.grid.removeNode(node);
+        this.grid.removeNode(node, detachNode);
         el.removeData('_gridstack_node');
         this._updateContainerHeight();
         if (detachNode) {