resize with sizeToContent

adumesny · adumesny · commit 8d233b0e65da · 2024-03-03T10:53:37.000-08:00
fix gridstack#2612 to allow resize when you have a number (soft max)
diff --git a/src/dd-gridstack.ts b/src/dd-gridstack.ts
@@ -46,7 +46,7 @@ export class DDGridStack {
         if (handles === 'all') handles = 'n,e,s,w,se,sw,ne,nw';
         // NOTE: keep the resize handles as e,w don't have enough space (10px) to show resize corners anyway. limit during drag instead
         // restrict vertical resize if height is done to match content anyway... odd to have it spring back
-        // if (Utils.shouldSizeToContent(n)) {
+        // if (Utils.shouldSizeToContent(n, true)) {
         //   const doE = handles.indexOf('e') !== -1;
         //   const doW = handles.indexOf('w') !== -1;
         //   handles = doE ? (doW ? 'e,w' : 'e') : (doW ? 'w' : '');
diff --git a/src/dd-resizable.ts b/src/dd-resizable.ts
@@ -166,7 +166,7 @@ export class DDResizable extends DDBaseImplement implements HTMLElementExtendOpt
 
   /** @internal */
   protected _resizeStart(event: MouseEvent): DDResizable {
-    this.sizeToContent = Utils.shouldSizeToContent(this.el.gridstackNode);
+    this.sizeToContent = Utils.shouldSizeToContent(this.el.gridstackNode, true); // strick true only and not number
     this.originalRect = this.el.getBoundingClientRect();
     this.scrollEl = Utils.getScrollElement(this.el);
     this.scrollY = this.scrollEl.scrollTop;
diff --git a/src/utils.ts b/src/utils.ts
@@ -109,9 +109,11 @@ export class Utils {
     return els;
   }
 
-  /** true if we should resize to content */
-  static shouldSizeToContent(n: GridStackNode | undefined): boolean {
-    return n?.grid && (!!n.sizeToContent || (n.grid.opts.sizeToContent && n.sizeToContent !== false));
+  /** true if we should resize to content. strict=true when only 'sizeToContent:true' and not a number which lets user adjust */
+  static shouldSizeToContent(n: GridStackNode | undefined, strict = false): boolean {
+    return n?.grid && (strict ? 
+    (n.sizeToContent === true || (n.grid.opts.sizeToContent === true && n.sizeToContent === undefined)) :
+    (!!n.sizeToContent || (n.grid.opts.sizeToContent && n.sizeToContent !== false)));
   }
 
   /** returns true if a and b overlap */

Original file line number	Diff line number	Diff line change
`@@ -109,9 +109,11 @@ export class Utils {`
`109`	`109`	`return els;`
`110`	`110`	`}`
`111`	`111`
`112`		`- /** true if we should resize to content */`
`113`		`- static shouldSizeToContent(n: GridStackNode \| undefined): boolean {`
`114`		`- return n?.grid && (!!n.sizeToContent \|\| (n.grid.opts.sizeToContent && n.sizeToContent !== false));`
	`112`	`+ /** true if we should resize to content. strict=true when only 'sizeToContent:true' and not a number which lets user adjust */`
	`113`	`+ static shouldSizeToContent(n: GridStackNode \| undefined, strict = false): boolean {`
	`114`	`+ return n?.grid && (strict ?`
	`115`	`+ (n.sizeToContent === true \|\| (n.grid.opts.sizeToContent === true && n.sizeToContent === undefined)) :`
	`116`	`+ (!!n.sizeToContent \|\| (n.grid.opts.sizeToContent && n.sizeToContent !== false)));`
`115`	`117`	`}`
`116`	`118`
`117`	`119`	`/** returns true if a and b overlap */`