[FIX] FiguresContainer: Restrict resizing if reaches header boundary

Resizing the graph previously allowed it to go beyond the header, causing the
entire graph to be pushed to the right or bottom upon release. This commit
addresses this issue by introducing a restriction that prevents resizing from
extending into the header.

Changes made:
- Added conditional checks to restrict resizing if the x or y coordinate reaches
the header boundaries.

closes #2726

Task: 3414127
X-original-commit: b1bac2f
Signed-off-by: Adrien Minne (adrm) <adrm@odoo.com>

Author: dhrp-odoo

src/components/figures/figure_container/figure_container.ts

@@ -287,6 +287,15 @@ export class FiguresContainer extends Component<Props, SpreadsheetChildEnv> {
         if (dirY < 0) {
           this.dnd.y = dndInitialY + figure.height * fraction;
         }
+
+        if (this.dnd.x < 0) {
+          this.dnd.width += this.dnd.x;
+          this.dnd.x = 0;
+        }
+        if (this.dnd.y < 0) {
+          this.dnd.height += this.dnd.y;
+          this.dnd.y = 0;
+        }
       };
     } else {
       onMouseMove = (ev: MouseEvent) => {
@@ -301,6 +310,15 @@ export class FiguresContainer extends Component<Props, SpreadsheetChildEnv> {
         if (dirY < 0) {
           this.dnd.y = dndInitialY - deltaY;
         }
+
+        if (this.dnd.x < 0) {
+          this.dnd.width += this.dnd.x;
+          this.dnd.x = 0;
+        }
+        if (this.dnd.y < 0) {
+          this.dnd.height += this.dnd.y;
+          this.dnd.y = 0;
+        }
       };
     }
     const onMouseUp = (ev: MouseEvent) => {

tests/components/figure.test.ts

@@ -267,6 +267,24 @@ describe("figures", () => {
     expect(model.getters.getFigure(sheetId, figureId)).toMatchObject(expectedSize);
   });
 
+  test.each([
+    ["top", { mouseOffsetX: 0, mouseOffsetY: -100 }],
+    ["left", { mouseOffsetX: -100, mouseOffsetY: 0 }],
+    ["topLeft", { mouseOffsetX: -100, mouseOffsetY: -100 }],
+  ])(
+    "Resizing a figure through its top and left anchor does not change size beyond header boundaries",
+    async (anchor: string, mouseMove: { mouseOffsetX: number; mouseOffsetY: number }) => {
+      const figureId = "someuuid";
+      const sheetId = model.getters.getActiveSheetId();
+      const figure = { width: 100, height: 100 };
+      createFigure(model, { id: figureId, y: 0, x: 0, ...figure });
+      await nextTick();
+      await simulateClick(".o-figure");
+      await dragAnchor(anchor, mouseMove.mouseOffsetX, mouseMove.mouseOffsetY, true);
+      expect(model.getters.getFigure(sheetId, figureId)).toMatchObject(figure);
+    }
+  );
+
   test.each([
     [
       "topLeft",