Merge pull request #3398 from adumesny/3012-resizeUpdate

* fix #3012 keep node._orig during a gesture so the stop event can't crash

Author: adumesny

doc/CHANGES.md

@@ -150,6 +150,7 @@ Change log
 ## 14.0.0 (TBD)
 * feat: [#754](https://github.com/gridstack/gridstack.js/issues/754) new `mode?: 'top' | 'float' | 'list' | 'compact'` - items are continuously re-flowed in sequential (row-major) order, like a re-orderable list: dragging, resizing, adding or removing an item re-flows everyone else instead of pushing them down, and dropping an item on another takes its place. See new
 [list.html](https://gridstackjs.com/demo/list.html) demo.
+* fix: [#3012](https://github.com/gridstack/gridstack.js/issues/3012) prevent crash when calling update() during change event
 
 ## 13.3.0 (2026-09-11)
 * feat: [#2781](https://github.com/gridstack/gridstack.js/issues/3177) [#2781](https://github.com/gridstack/gridstack.js/issues/3177) mobile: pause to drag/reszie vs scroll behavior

spec/regression-spec.ts

@@ -256,4 +256,41 @@ describe('regression >', () => {
       grid.engine.endUpdate();
     });
   });
+
+  describe('3012 update() from inside a drag/resize handler >', () => {
+    beforeEach(() => {
+      document.body.insertAdjacentHTML('afterbegin', gridstackEmptyHTML);
+    });
+    afterEach(() => {
+      document.body.removeChild(document.getElementById('gs-cont'));
+    });
+
+    it('keeps node._orig alive so the stop event does not crash', () => {
+      grid = GridStack.init({cellHeight: 50, children: [{id: 'A', x: 0, y: 0, w: 3, h: 3}]});
+      const A = grid.engine.nodes.find(n => n.id === 'A')!;
+
+      grid.engine.cleanNodes().beginUpdate(A); // resizestart
+      expect(A._updating).toBe(true);
+      expect(A._orig).toEqual({x: 0, y: 0, w: 3, h: 3});
+
+      // the aspect-ratio handler: update() from inside the 'resize' event
+      grid.update(A.el!, {h: 4});
+      expect(A.h).toBe(4);
+      // used to be deleted here, then onEndMoving did `node._orig!.w` -> TypeError
+      expect(A._orig).toEqual({x: 0, y: 0, w: 3, h: 3});
+      expect(() => A.w !== A._orig!.w).not.toThrow();
+
+      grid.engine.endUpdate();
+    });
+
+    it('still clears _orig for a plain update() outside a gesture (#2669)', () => {
+      grid = GridStack.init({cellHeight: 50, children: [{id: 'A', x: 0, y: 0, w: 3, h: 3}]});
+      const A = grid.engine.nodes.find(n => n.id === 'A')!;
+      grid.engine.saveInitial();
+      expect(A._orig).toBeDefined();
+      grid.update(A.el!, {h: 4});
+      expect(A._updating).toBeFalsy();
+      expect(A._orig).toBeUndefined();
+    });
+  });
 });

src/gridstack.ts

@@ -1653,7 +1653,10 @@ export class GridStack {
         } else {
           this.resizeToContentCheck(widthChanged, n);
         }
-        delete n._orig; // clear out original position now that we moved #2669
+        // clear out original position now that we moved #2669 - but NOT mid drag/resize, where _orig is
+        // the gesture's baseline that _dragOrResize/onEndMoving still need. A user calling update() from
+        // a 'drag'/'resize' handler used to delete it and crash on the following stop event (#3012).
+        if (!n._updating) delete n._orig;
       }
       if (m || changed) {
         this._writeAttr(el, n);
@@ -2919,7 +2922,7 @@ export class GridStack {
         delete node._resizing;
         delete node._event;
         delete node._lastTried;
-        const widthChanged = node.w !== node._orig!.w;
+        const widthChanged = !!node._orig && node.w !== node._orig.w;
 
         // if the item has moved to another grid, we're done here
         const target: GridItemHTMLElement = event.target as GridItemHTMLElement;