@excalidraw/excalidraw 0.18.0-rc.3 → 0.18.0-rc.4

package/README.md

@@ -16,7 +16,7 @@ yarn add react react-dom @excalidraw/excalidraw
 
 #### Self-hosting fonts
 
-By default, excalidraw will try to load fonts used by Excalidraw from the [CDN](https://esm.run/@excalidraw/excalidraw/dist/prod).
+By default, Excalidraw will try to download all the used fonts from the [CDN](https://esm.run/@excalidraw/excalidraw/dist/prod).
 
 For self-hosting purposes, you'll have to copy the content of the folder `node_modules/@excalidraw/excalidraw/dist/prod/fonts` to the path where your assets should be served from (i.e. `public/` directory in your project). In that case, you should also set `window.EXCALIDRAW_ASSET_PATH` to the very same path, i.e. `/` in case it's in the root:
 
@@ -30,7 +30,8 @@ Excalidraw takes _100%_ of `width` and `height` of the containing block so make
 
 ## Demo
 
-Go to [CodeSandbox](https://codesandbox.io/s/excalidraw-ehlz3).
+<!-- // TODO_V18: temp -->
+Go to [CodeSandbox](https://codesandbox.io/p/sandbox/github/excalidraw/excalidraw/tree/mrazator/release-v18/examples/with-script-in-browser) example.
 
 ## Integration
 

package/dist/dev/{chunk-CI27YPGF.js → chunk-3O3BNMYV.js}

@@ -14726,6 +14726,16 @@ var _newElementBase = (type, {
   locked = DEFAULT_ELEMENT_PROPS.locked,
   ...rest
 }) => {
+  if (x < -1e6 || x > 1e6 || y < -1e6 || y > 1e6 || width < -1e6 || width > 1e6 || height < -1e6 || height > 1e6) {
+    console.error("New element size or position is too large", {
+      x,
+      y,
+      width,
+      height,
+      // @ts-ignore
+      points: rest.points
+    });
+  }
   const element = {
     id: rest.id || randomId(),
     type,
@@ -18660,6 +18670,12 @@ var restoreElementWithProperties = (element, extra) => {
   if ("customData" in element || "customData" in extra) {
     base.customData = "customData" in extra ? extra.customData : element.customData;
   }
+  if (element.x < -1e6 || element.x > 1e6 || element.y < -1e6 || element.y > 1e6 || element.width < -1e6 || element.width > 1e6 || element.height < -1e6 || element.height > 1e6) {
+    console.error(
+      "Restore element with properties size or position is too large",
+      { element }
+    );
+  }
   return {
     // spread the original element properties to not lose unknown ones
     // for forward-compatibility
@@ -18671,6 +18687,9 @@ var restoreElementWithProperties = (element, extra) => {
   };
 };
 var restoreElement = (element) => {
+  if (element.x < -1e6 || element.x > 1e6 || element.y < -1e6 || element.y > 1e6 || element.width < -1e6 || element.width > 1e6 || element.height < -1e6 || element.height > 1e6) {
+    console.error("Restore element size or position is too large", { element });
+  }
   switch (element.type) {
     case "text":
       let fontSize = element.fontSize;
@@ -20979,7 +20998,7 @@ var BinaryHeap = class {
 var DEDUP_TRESHOLD = 1;
 var BASE_PADDING = 40;
 var handleSegmentRenormalization = (arrow, elementsMap) => {
-  const nextFixedSegments = arrow.fixedSegments ? structuredClone(arrow.fixedSegments) : null;
+  const nextFixedSegments = arrow.fixedSegments ? arrow.fixedSegments.slice() : null;
   if (nextFixedSegments) {
     const _nextPoints = [];
     arrow.points.map((p) => pointFrom(arrow.x + p[0], arrow.y + p[1])).forEach((p, i, points) => {
@@ -21505,10 +21524,30 @@ var handleEndpointDrag = (arrow, updatedPoints, fixedSegments, startHeading, end
     endIsSpecial
   );
 };
+var MAX_POS = 1e6;
 var updateElbowArrowPoints = (arrow, elementsMap, updates, options) => {
   if (arrow.points.length < 2) {
     return { points: updates.points ?? arrow.points };
   }
+  if (arrow.x < -MAX_POS || arrow.x > MAX_POS || arrow.y < -MAX_POS || arrow.y > MAX_POS || arrow.x + (updates?.points?.[updates?.points?.length - 1]?.[0] ?? 0) < -MAX_POS || arrow.x + (updates?.points?.[updates?.points?.length - 1]?.[0] ?? 0) > MAX_POS || arrow.y + (updates?.points?.[updates?.points?.length - 1]?.[1] ?? 0) < -MAX_POS || arrow.y + (updates?.points?.[updates?.points?.length - 1]?.[1] ?? 0) > MAX_POS || arrow.x + (arrow?.points?.[arrow?.points?.length - 1]?.[0] ?? 0) < -MAX_POS || arrow.x + (arrow?.points?.[arrow?.points?.length - 1]?.[0] ?? 0) > MAX_POS || arrow.y + (arrow?.points?.[arrow?.points?.length - 1]?.[1] ?? 0) < -MAX_POS || arrow.y + (arrow?.points?.[arrow?.points?.length - 1]?.[1] ?? 0) > MAX_POS) {
+    console.error(
+      "Elbow arrow (or update) is outside reasonable bounds (> 1e6)",
+      {
+        arrow,
+        updates
+      }
+    );
+  }
+  arrow.x = clamp(arrow.x, -MAX_POS, MAX_POS);
+  arrow.y = clamp(arrow.y, -MAX_POS, MAX_POS);
+  if (updates.points) {
+    updates.points = updates.points.map(
+      ([x, y]) => pointFrom(
+        clamp(x, -MAX_POS, MAX_POS),
+        clamp(y, -MAX_POS, MAX_POS)
+      )
+    );
+  }
   if (!define_import_meta_env_default.PROD) {
     invariant(
       !updates.points || updates.points.length >= 2,
@@ -21541,7 +21580,7 @@ var updateElbowArrowPoints = (arrow, elementsMap, updates, options) => {
   }
   const updatedPoints = updates.points ? updates.points && updates.points.length === 2 ? arrow.points.map(
     (p, idx) => idx === 0 ? updates.points[0] : idx === arrow.points.length - 1 ? updates.points[1] : p
-  ) : structuredClone(updates.points) : structuredClone(arrow.points);
+  ) : updates.points.slice() : arrow.points.slice();
   const {
     startHeading,
     endHeading,
@@ -22189,16 +22228,30 @@ var getBindableElementForId = (id, elementsMap) => {
 var normalizeArrowElementUpdate = (global, nextFixedSegments, startIsSpecial, endIsSpecial) => {
   const offsetX = global[0][0];
   const offsetY = global[0][1];
-  const points = global.map(
+  let points = global.map(
     (p) => pointTranslate(
       p,
       vectorScale(vectorFromPoint(global[0]), -1)
     )
   );
+  if (offsetX < -MAX_POS || offsetX > MAX_POS || offsetY < -MAX_POS || offsetY > MAX_POS || offsetX + points[points.length - 1][0] < -MAX_POS || offsetY + points[points.length - 1][0] > MAX_POS || offsetX + points[points.length - 1][1] < -MAX_POS || offsetY + points[points.length - 1][1] > MAX_POS) {
+    console.error(
+      "Elbow arrow normalization is outside reasonable bounds (> 1e6)",
+      {
+        x: offsetX,
+        y: offsetY,
+        points,
+        ...getSizeFromPoints(points)
+      }
+    );
+  }
+  points = points.map(
+    ([x, y]) => pointFrom(clamp(x, -1e6, 1e6), clamp(y, -1e6, 1e6))
+  );
   return {
     points,
-    x: offsetX,
-    y: offsetY,
+    x: clamp(offsetX, -1e6, 1e6),
+    y: clamp(offsetY, -1e6, 1e6),
     fixedSegments: (nextFixedSegments?.length ?? 0) > 0 ? nextFixedSegments : null,
     ...getSizeFromPoints(points),
     startIsSpecial,
@@ -23436,9 +23489,19 @@ var getResizedOrigin = (prevOrigin, prevWidth, prevHeight, newWidth, newHeight,
         y: y - (newHeight - prevHeight) / 2
       };
     case "east-side":
+      if (Math.abs(
+        y + (prevWidth - newWidth) / 2 * Math.sin(angle) + (prevHeight - newHeight) / 2
+      ) > 1e6) {
+        console.error(
+          "getResizedOrigin() new calculation creates extremely large (> 1e6) y value where the old calculation resulted in",
+          {
+            result: y + (newHeight - prevHeight) / 2 + (prevWidth - newWidth) / 2 * Math.sin(angle)
+          }
+        );
+      }
       return {
         x: x + (prevWidth - newWidth) / 2 * (Math.cos(angle) + 1),
-        y: y + (prevWidth - newWidth) / 2 * Math.sin(angle) + (prevHeight - newHeight) / 2
+        y: y + (newHeight - prevHeight) / 2 + (prevWidth - newWidth) / 2 * Math.sin(angle)
       };
     case "west-side":
       return {
@@ -24179,7 +24242,7 @@ var parseFileContents = async (blob) => {
   let contents;
   if (blob.type === MIME_TYPES.png) {
     try {
-      return await (await import("./data/image-TC7JM54G.js")).decodePngMetadata(blob);
+      return await (await import("./data/image-EN763OZS.js")).decodePngMetadata(blob);
     } catch (error) {
       if (error.message === "INVALID") {
         throw new ImageSceneDataError(
@@ -25253,4 +25316,4 @@ export {
   getNormalizedZoom,
   getNormalizedGridStep
 };
-//# sourceMappingURL=chunk-CI27YPGF.js.map
+//# sourceMappingURL=chunk-3O3BNMYV.js.map