gridstack 9.4.0 → 9.5.1

package/dist/gridstack.js

@@ -1,5 +1,5 @@
 /*!
- * GridStack 9.4.0
+ * GridStack 9.5.1
  * https://gridstackjs.com/
  *
  * Copyright (c) 2021-2022 Alain Dumesny
@@ -551,7 +551,8 @@ class GridStack {
         this._insertNotAppend = haveCoord; // if we create in reverse order...
         // if we're loading a layout into for example 1 column (_prevColumn is set only when going to 1) and items don't fit, make sure to save
         // the original wanted layout so we can scale back up correctly #1471
-        if (this._prevColumn && this._prevColumn !== this.opts.column && items.some(n => ((n.x || 0) + n.w) > this.opts.column)) {
+        const column = this.opts.column;
+        if (this._prevColumn && this._prevColumn !== column && items.some(n => ((n.x || 0) + n.w) > column)) {
             this._ignoreLayoutsNodeChange = true; // skip layout update
             this.engine.cacheLayout(items, this._prevColumn, true);
         }
@@ -594,6 +595,7 @@ class GridStack {
                 if (Utils.shouldSizeToContent(item))
                     w.h = item.h;
                 // check if missing coord, in which case find next empty slot with new (or old if missing) sizes
+                this.engine.nodeBoundFix(w); // before widthChanged is checked below...
                 if (w.autoPosition || w.x === undefined || w.y === undefined) {
                     w.w = w.w || item.w;
                     w.h = w.h || item.h;
@@ -604,7 +606,6 @@ class GridStack {
                 this.engine.nodes.push(item);
                 if (Utils.samePos(item, w)) {
                     this.moveNode(item, { ...w, forceCollide: true });
-                    Utils.copyPos(w, item, true);
                 }
                 this.update(item.el, w);
                 if (w.subGridOpts?.children) { // update any sub grid as well
@@ -738,9 +739,9 @@ class GridStack {
         if (!column || column < 1 || this.opts.column === column)
             return this;
         let oldColumn = this.getColumn();
-        // if we go into 1 column mode (which happens if we're sized less than minW unless disableOneColumnMode is on)
+        // if we go into 1 column mode due to size change (disableOneColumnMode is off and we hit min width)
         // then remember the original columns so we can restore.
-        if (column === 1) {
+        if (column === 1 && !this.opts.disableOneColumnMode) {
             this._prevColumn = oldColumn;
         }
         else {
@@ -1077,6 +1078,7 @@ class GridStack {
             if (!n)
                 return;
             let w = Utils.cloneDeep(opt); // make a copy we can modify in case they re-use it or multiple items
+            this.engine.nodeBoundFix(w);
             delete w.autoPosition;
             delete w.id;
             // move/resize widget if anything changed
@@ -1118,10 +1120,10 @@ class GridStack {
                 }
             }
             Utils.sanitizeMinMax(n);
-            // finally move the widget
-            if (m !== undefined)
+            // finally move the widget and update attr
+            if (m)
                 this.moveNode(n, m);
-            if (changed) { // move will only update x,y,w,h so update the rest too
+            if (m || changed) {
                 this._writeAttr(el, n);
             }
             if (ddChanged) {
@@ -1554,6 +1556,8 @@ class GridStack {
         // update any gridItem height with sizeToContent, but wait for DOM $animation_speed to settle if we changed column count
         // TODO: is there a way to know what the final (post animation) size of the content will be so we can animate the column width and height together rather than sequentially ?
         setTimeout(() => {
+            if (!this.engine)
+                return; // we've been deleted in between!
             if (n) {
                 if (Utils.shouldSizeToContent(n))
                     this.resizeToContentCheck(n.el, useAttr);
@@ -1980,6 +1984,7 @@ class GridStack {
                 this.engine.cleanupNode(node); // removes all internal _xyz values
                 node.grid = this;
             }
+            delete node.grid._isTemp;
             dd.off(el, 'drag');
             // if we made a copy ('helper' which is temp) of the original node then insert a copy, else we move the original node (#1102)
             // as the helper will be nuked by jquery-ui otherwise. TODO: update old code path
@@ -2010,6 +2015,7 @@ class GridStack {
                 if (!subGrid.opts.styleInHead)
                     subGrid._updateStyles(true); // re-create sub-grid styles now that we've moved
             }
+            this._prepareDragDropByNode(node);
             this._updateContainerHeight();
             this.engine.addedNodes.push(node); // @ts-ignore
             this._triggerAddEvent(); // @ts-ignore
@@ -2018,17 +2024,6 @@ class GridStack {
             if (this._gsEventHandler['dropped']) {
                 this._gsEventHandler['dropped']({ ...event, type: 'dropped' }, origNode && origNode.grid ? origNode : undefined, node);
             }
-            // wait till we return out of the drag callback to set the new drag&resize handler or they may get messed up
-            window.setTimeout(() => {
-                // IFF we are still there (some application will use as placeholder and insert their real widget instead and better call makeWidget())
-                if (node.el && node.el.parentElement) {
-                    this._prepareDragDropByNode(node);
-                }
-                else {
-                    this.engine.removeNode(node);
-                }
-                delete node.grid._isTemp;
-            });
             return false; // prevent parent from receiving msg (which may be grid as well)
         });
         return this;
@@ -2310,6 +2305,6 @@ GridStack.resizeToContentParent = '.grid-stack-item-content';
 GridStack.Utils = Utils;
 /** scoping so users can call new GridStack.Engine(12) for example */
 GridStack.Engine = GridStackEngine;
-GridStack.GDRev = '9.4.0';
+GridStack.GDRev = '9.5.1';
 export { GridStack };
 //# sourceMappingURL=gridstack.js.map