fix #267

Author: troolee

README.md

@@ -828,6 +828,7 @@ Changes
 - add `enableMove`/`enableResize` methods.
 - fix window resize issue #331.
 - add options `disableDrag` and `disableResize`.
+- fix `batchUpdate`/`commit` (Thank to @radiolips)
 
 #### v0.2.4 (2016-02-15)
 

dist/gridstack.js

@@ -131,8 +131,8 @@
     };
 
     GridStackEngine.prototype.commit = function() {
-        this._updateCounter = 0;
-        if (this._updateCounter === 0) {
+        if (this._updateCounter !== 0) {
+            this._updateCounter = 0;
             this.float = this._float;
             this._packNodes();
             this._notify();
@@ -269,6 +269,9 @@
     };
 
     GridStackEngine.prototype.cleanNodes = function() {
+        if (this._updateCounter) {
+            return;
+        }
         _.each(this.nodes, function(n) {n._dirty = false; });
     };
 

dist/gridstack.min.js

@@ -131,8 +131,8 @@
     };
 
     GridStackEngine.prototype.commit = function() {
-        this._updateCounter = 0;
-        if (this._updateCounter === 0) {
+        if (this._updateCounter !== 0) {
+            this._updateCounter = 0;
             this.float = this._float;
             this._packNodes();
             this._notify();
@@ -269,6 +269,9 @@
     };
 
     GridStackEngine.prototype.cleanNodes = function() {
+        if (this._updateCounter) {
+            return;
+        }
         _.each(this.nodes, function(n) {n._dirty = false; });
     };