gridstack 10.1.1 → 10.2.0

package/dist/gridstack.js

@@ -1,5 +1,5 @@
 /*!
- * GridStack 10.1.1
+ * GridStack 10.2.0
  * https://gridstackjs.com/
  *
  * Copyright (c) 2021-2022 Alain Dumesny
@@ -218,7 +218,8 @@ class GridStack {
         if (bk?.length > 1)
             bk.sort((a, b) => (b.w || 0) - (a.w || 0));
         // elements DOM attributes override any passed options (like CSS style) - merge the two together
-        let defaults = { ...Utils.cloneDeep(gridDefaults),
+        let defaults = {
+            ...Utils.cloneDeep(gridDefaults),
             column: Utils.toNumber(el.getAttribute('gs-column')) || gridDefaults.column,
             minRow: rowAttr ? rowAttr : Utils.toNumber(el.getAttribute('gs-min-row')) || gridDefaults.minRow,
             maxRow: rowAttr ? rowAttr : Utils.toNumber(el.getAttribute('gs-max-row')) || gridDefaults.maxRow,
@@ -313,7 +314,7 @@ class GridStack {
                 this.load(children); // don't load empty
         }
         // if (this.engine.nodes.length) this._updateStyles(); // update based on # of children. done in engine onChange CB
-        this.setAnimation(opts.animate);
+        this.setAnimation();
         // dynamic grids require pausing during drag to detect over to nest vs push
         if (opts.subGridDynamic && !DDManager.pauseDrag)
             DDManager.pauseDrag = true;
@@ -585,10 +586,17 @@ class GridStack {
     load(items, addRemove = GridStack.addRemoveCB || true) {
         items = Utils.cloneDeep(items); // so we can mod
         const column = this.getColumn();
+        // if we have a mix of new items without coordinates and existing items, separate them out so they can be added after #2639
+        let addAfter = items.filter(n => (n.x === undefined || n.y === undefined) && !Utils.find(this.engine.nodes, n.id));
+        if (addAfter.length && addAfter.length !== items.length) {
+            items = items.filter(n => !Utils.find(addAfter, n.id));
+        }
+        else
+            addAfter = [];
         // if passed list has coordinates, use them (insert from end to beginning for conflict resolution) else keep widget order
         const haveCoord = items.some(w => w.x !== undefined || w.y !== undefined);
         if (haveCoord)
-            items = Utils.sort(items, -1, column);
+            items = Utils.sort(items, -1);
         this._insertNotAppend = haveCoord; // if we create in reverse order...
         // if we're loading a layout into for example 1 column and items don't fit, make sure to save
         // the original wanted layout so we can scale back up correctly #1471
@@ -662,6 +670,10 @@ class GridStack {
                 this.addWidget(w);
             }
         });
+        // finally append any separate ones that didn't have explicit coordinates last so they can find next empty spot
+        if (addRemove) {
+            addAfter.forEach(w => this.addWidget(w));
+        }
         this.engine.removedNodes = removed;
         this.batchUpdate(false);
         // after commit, clear that flag
@@ -669,8 +681,8 @@ class GridStack {
         delete this._insertNotAppend;
         prevCB ? GridStack.addRemoveCB = prevCB : delete GridStack.addRemoveCB;
         // delay adding animation back
-        if (noAnim && this.opts.animate)
-            setTimeout(() => this.setAnimation(this.opts.animate));
+        if (noAnim && this.opts?.animate)
+            this.setAnimation(this.opts.animate, true);
         return this;
     }
     /**
@@ -1042,7 +1054,7 @@ class GridStack {
             }
             if (!node)
                 return;
-            if (GridStack.addRemoveCB) {
+            if (removeDOM && GridStack.addRemoveCB) {
                 GridStack.addRemoveCB(this.el, node, false, false);
             }
             // remove our DOM data (circular link) and drag&drop permanently
@@ -1062,23 +1074,35 @@ class GridStack {
     /**
      * Removes all widgets from the grid.
      * @param removeDOM if `false` DOM elements won't be removed from the tree (Default? `true`).
+     * @param triggerEvent if `false` (quiet mode) element will not be added to removed list and no 'removed' callbacks will be called (Default? true).
      */
-    removeAll(removeDOM = true) {
+    removeAll(removeDOM = true, triggerEvent = true) {
         // always remove our DOM data (circular link) before list gets emptied and drag&drop permanently
         this.engine.nodes.forEach(n => {
+            if (removeDOM && GridStack.addRemoveCB) {
+                GridStack.addRemoveCB(this.el, n, false, false);
+            }
             delete n.el.gridstackNode;
-            this._removeDD(n.el);
+            if (!this.opts.staticGrid)
+                this._removeDD(n.el);
         });
-        this.engine.removeAll(removeDOM);
-        this._triggerRemoveEvent();
+        this.engine.removeAll(removeDOM, triggerEvent);
+        if (triggerEvent)
+            this._triggerRemoveEvent();
         return this;
     }
     /**
      * Toggle the grid animation state.  Toggles the `grid-stack-animate` class.
      * @param doAnimate if true the grid will animate.
+     * @param delay if true setting will be set on next event loop.
      */
-    setAnimation(doAnimate) {
-        if (doAnimate) {
+    setAnimation(doAnimate = this.opts.animate, delay) {
+        if (delay) {
+            // delay, but check to make sure grid (opt) is still around
+            setTimeout(() => { if (this.opts)
+                this.setAnimation(doAnimate); });
+        }
+        else if (doAnimate) {
             this.el.classList.add('grid-stack-animate');
         }
         else {
@@ -1188,12 +1212,15 @@ class GridStack {
         return this;
     }
     moveNode(n, m) {
-        this.engine.cleanNodes()
-            .beginUpdate(n)
-            .moveNode(n, m);
+        const wasUpdating = n._updating;
+        if (!wasUpdating)
+            this.engine.cleanNodes().beginUpdate(n);
+        this.engine.moveNode(n, m);
         this._updateContainerHeight();
-        this._triggerChangeEvent();
-        this.engine.endUpdate();
+        if (!wasUpdating) {
+            this._triggerChangeEvent();
+            this.engine.endUpdate();
+        }
     }
     /**
      * Updates widget height to match the content height to avoid v-scrollbar or dead space.
@@ -1267,6 +1294,27 @@ class GridStack {
         else
             this.resizeToContent(el);
     }
+    /** rotate (by swapping w & h) the passed in node - called when user press 'r' during dragging
+     * @param els  widget or selector of objects to modify
+     * @param relative optional pixel coord relative to upper/left corner to rotate around (will keep that cell under cursor)
+     */
+    rotate(els, relative) {
+        GridStack.getElements(els).forEach(el => {
+            let n = el.gridstackNode;
+            if (!n || n.w === n.h)
+                return;
+            const rot = { w: n.h, h: n.w, minH: n.minW, minW: n.minH, maxH: n.maxW, maxW: n.maxH };
+            // if given an offset, adjust x/y by column/row bounds when user presses 'r' during dragging
+            if (relative) {
+                let pivotX = relative.left > 0 ? Math.floor(relative.left / this.cellWidth()) : 0;
+                let pivotY = relative.top > 0 ? Math.floor(relative.top / this.opts.cellHeight) : 0;
+                rot.x = n.x + pivotX - (n.h - (pivotY + 1));
+                rot.y = (n.y + pivotY) - pivotX;
+            }
+            this.update(el, rot);
+        });
+        return this;
+    }
     /**
      * Updates the margins which will set all 4 sides at once - see `GridStackOptions.margin` for format options (CSS string format of 1,2,4 values or single number).
      * @param value margin value
@@ -1334,8 +1382,9 @@ class GridStack {
             }
             // prevent added nodes from also triggering 'change' event (which is called next)
             this.engine.addedNodes.forEach(n => { delete n._dirty; });
-            this._triggerEvent('added', this.engine.addedNodes);
+            const addedNodes = [...this.engine.addedNodes];
             this.engine.addedNodes = [];
+            this._triggerEvent('added', addedNodes);
         }
         return this;
     }
@@ -1344,8 +1393,9 @@ class GridStack {
         if (this.engine.batchMode)
             return this;
         if (this.engine.removedNodes?.length) {
-            this._triggerEvent('removed', this.engine.removedNodes);
+            const removedNodes = [...this.engine.removedNodes];
             this.engine.removedNodes = [];
+            this._triggerEvent('removed', removedNodes);
         }
         return this;
     }
@@ -1917,7 +1967,7 @@ class GridStack {
         };
         dd.droppable(this.el, {
             accept: (el) => {
-                let node = el.gridstackNode;
+                let node = el.gridstackNode || this._readAttr(el, false);
                 // set accept drop to true on ourself (which we ignore) so we don't get "can't drop" icon in HTML5 mode while moving
                 if (node?.grid === this)
                     return true;
@@ -1932,7 +1982,7 @@ class GridStack {
                     let selector = (this.opts.acceptWidgets === true ? '.grid-stack-item' : this.opts.acceptWidgets);
                     canAccept = el.matches(selector);
                 }
-                // finally check to make sure we actually have space left #1571
+                // finally check to make sure we actually have space left #1571 #2633
                 if (canAccept && node && this.opts.maxRow) {
                     let n = { w: node.w, h: node.h, minW: node.minW, minH: node.minH }; // only width/height matters and autoPosition
                     canAccept = this.engine.willItFit(n);
@@ -1994,7 +2044,7 @@ class GridStack {
                 node._temporaryRemoved = true; // so we can insert it
             }
             // clear any marked for complete removal (Note: don't check _isAboutToRemove as that is cleared above - just do it)
-            this._itemRemoving(node.el, false);
+            GridStack._itemRemoving(node.el, false);
             dd.on(el, 'drag', onDrag);
             // make sure this is called at least once when going fast #1578
             onDrag(event, el, helper);
@@ -2054,7 +2104,7 @@ class GridStack {
                 this.engine.cleanupNode(node); // removes all internal _xyz values
                 node.grid = this;
             }
-            delete node.grid._isTemp;
+            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
@@ -2094,33 +2144,33 @@ class GridStack {
             }
             // delay adding animation back
             if (noAnim)
-                setTimeout(() => this.setAnimation(this.opts.animate));
+                this.setAnimation(this.opts.animate, true);
             return false; // prevent parent from receiving msg (which may be grid as well)
         });
         return this;
     }
     /** @internal mark item for removal */
-    _itemRemoving(el, remove) {
-        let node = el ? el.gridstackNode : undefined;
-        if (!node || !node.grid || el.classList.contains(this.opts.removableOptions.decline))
+    static _itemRemoving(el, remove) {
+        const node = el ? el.gridstackNode : undefined;
+        if (!node?.grid || el.classList.contains(node.grid.opts.removableOptions.decline))
             return;
         remove ? node._isAboutToRemove = true : delete node._isAboutToRemove;
         remove ? el.classList.add('grid-stack-item-removing') : el.classList.remove('grid-stack-item-removing');
     }
     /** @internal called to setup a trash drop zone if the user specifies it */
     _setupRemoveDrop() {
-        if (!this.opts.staticGrid && typeof this.opts.removable === 'string') {
-            let trashEl = document.querySelector(this.opts.removable);
-            if (!trashEl)
-                return this;
-            // only register ONE drop-over/dropout callback for the 'trash', and it will
-            // update the passed in item and parent grid because the 'trash' is a shared resource anyway,
-            // and Native DD only has 1 event CB (having a list and technically a per grid removableOptions complicates things greatly)
-            if (!dd.isDroppable(trashEl)) {
-                dd.droppable(trashEl, this.opts.removableOptions)
-                    .on(trashEl, 'dropover', (event, el) => this._itemRemoving(el, true))
-                    .on(trashEl, 'dropout', (event, el) => this._itemRemoving(el, false));
-            }
+        if (typeof this.opts.removable !== 'string')
+            return this;
+        let trashEl = document.querySelector(this.opts.removable);
+        if (!trashEl)
+            return this;
+        // only register ONE static drop-over/dropout callback for the 'trash', and it will
+        // update the passed in item and parent grid because the '.trash' is a shared resource anyway,
+        // and Native DD only has 1 event CB (having a list and technically a per grid removableOptions complicates things greatly)
+        if (!this.opts.staticGrid && !dd.isDroppable(trashEl)) {
+            dd.droppable(trashEl, this.opts.removableOptions)
+                .on(trashEl, 'dropover', (event, el) => GridStack._itemRemoving(el, true))
+                .on(trashEl, 'dropout', (event, el) => GridStack._itemRemoving(el, false));
         }
         return this;
     }
@@ -2226,6 +2276,7 @@ class GridStack {
         // @ts-ignore
         this._writePosAttr(this.placeholder, node);
         this.el.appendChild(this.placeholder);
+        this.placeholder.gridstackNode = node;
         // console.log('_onStartMoving placeholder') // TEST
         // if the element is inside a grid, it has already been scaled
         // we can use that as a scale reference
@@ -2257,11 +2308,11 @@ class GridStack {
             this.engine.addNode(node); // will add, fix collisions, update attr and clear _temporaryRemoved
             node._moving = true; // AFTER, mark as moving object (wanted fix location before)
         }
-        // set the min/max resize info
+        // set the min/max resize info taking into account the column count and position (so we don't resize outside the grid)
         this.engine.cacheRects(cellWidth, cellHeight, this.opts.marginTop, this.opts.marginRight, this.opts.marginBottom, this.opts.marginLeft);
         if (event.type === 'resizestart') {
-            dd.resizable(el, 'option', 'minWidth', cellWidth * (node.minW || 1))
-                .resizable(el, 'option', 'minHeight', cellHeight * (node.minH || 1));
+            dd.resizable(el, 'option', 'minWidth', cellWidth * Math.min(node.minW || 1, this.getColumn() - node.x))
+                .resizable(el, 'option', 'minHeight', cellHeight * Math.min(node.minH || 1, (this.opts.maxRow || Number.MAX_SAFE_INTEGER) - node.y));
             if (node.maxW) {
                 dd.resizable(el, 'option', 'maxWidth', cellWidth * node.maxW);
             }
@@ -2375,7 +2426,7 @@ class GridStack {
         node.el = node._isExternal && helper ? helper : el; // point back to real item being dragged
         if (this.opts.removable === true) { // boolean vs a class string
             // item leaving us and we are supposed to remove on leave (no need to drag onto trash) mark it so
-            this._itemRemoving(el, true);
+            GridStack._itemRemoving(el, true);
         }
         // finally if item originally came from another grid, but left us, restore things back to prev info
         if (el._gridstackNodeOrig) {
@@ -2400,6 +2451,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 = '10.1.1';
+GridStack.GDRev = '10.2.0';
 export { GridStack };
 //# sourceMappingURL=gridstack.js.map