Fix bug in bounds (#1009)

Author: wcandillon

package/src/skia/__tests__/Geometry.spec.ts

@@ -0,0 +1,15 @@
+import { importSkia } from "../../renderer/__tests__/setup";
+
+describe("Geometry", () => {
+  it("should calculate the bounding box the union of multiple rectangles", () => {
+    const { bounds, rect } = importSkia();
+
+    const r1 = rect(-10, -10, 30, 30);
+    const r2 = rect(30, 30, 100, 20);
+    const result = bounds([r1, r2]);
+    expect(result.x).toBe(-10);
+    expect(result.y).toBe(-10);
+    expect(result.width).toBe(140);
+    expect(result.height).toBe(60);
+  });
+});

package/src/skia/core/Rect.ts

@@ -12,7 +12,7 @@ export const bounds = (rects: SkRect[]) => {
   const y = Math.min(...rects.map((r) => r.y));
   const width = Math.max(...rects.map((r) => r.x + r.width));
   const height = Math.max(...rects.map((r) => r.y + r.height));
-  return rect(x, y, width, height);
+  return rect(x, y, width - x, height - y);
 };
 
 export const topLeft = (r: SkRect | SkRRect) =>