Merge pull request #655 from mathuo/654-resize

bug: fix layout force flag

Author: mathuo

packages/dockview-core/src/__tests__/gridview/baseComponentGridview.spec.ts

@@ -66,6 +66,8 @@ class TestPanel implements IGridPanelView {
 }
 
 class ClassUnderTest extends BaseGrid<TestPanel> {
+    readonly gridview = this.gridview;
+
     constructor(options: BaseGridOptions) {
         super(options);
     }
@@ -103,6 +105,31 @@ class ClassUnderTest extends BaseGrid<TestPanel> {
 }
 
 describe('baseComponentGridview', () => {
+    test('that .layout(...) force flag works', () => {
+        const cut = new ClassUnderTest({
+            parentElement: document.createElement('div'),
+            orientation: Orientation.HORIZONTAL,
+            proportionalLayout: true,
+        });
+
+        const spy = jest.spyOn(cut.gridview, 'layout');
+
+        cut.layout(100, 100);
+        expect(spy).toHaveBeenCalledTimes(1);
+
+        cut.layout(100, 100, false);
+        expect(spy).toHaveBeenCalledTimes(1);
+
+        cut.layout(100, 100, true);
+        expect(spy).toHaveBeenCalledTimes(2);
+
+        cut.layout(150, 150, false);
+        expect(spy).toHaveBeenCalledTimes(3);
+
+        cut.layout(150, 150, true);
+        expect(spy).toHaveBeenCalledTimes(4);
+    });
+
     test('can add group', () => {
         const cut = new ClassUnderTest({
             parentElement: document.createElement('div'),

packages/dockview-core/src/gridview/baseComponentGridview.ts

@@ -323,7 +323,7 @@ export abstract class BaseGrid<T extends IGridPanelView>
 
     public layout(width: number, height: number, forceResize?: boolean): void {
         const different =
-            forceResize ?? (width !== this.width || height !== this.height);
+            forceResize || width !== this.width || height !== this.height;
 
         if (!different) {
             return;