feat: do not strip unknown element properties on restore (excalidraw#…

…8682)
em3ndez · Oct 21, 2024 · 4cedf3d · 4cedf3d
1 parent e957c8e
commit 4cedf3d
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 147 deletions.
diff --git a/packages/excalidraw/data/restore.ts b/packages/excalidraw/data/restore.ts
@@ -190,6 +190,10 @@ const restoreElementWithProperties = <
   }
 
   return {
+    // spread the original element properties to not lose unknown ones
+    // for forward-compatibility
+    ...element,
+    // normalized properties
     ...base,
     ...getNormalizedDimensions(base),
     ...extra,

diff --git a/packages/excalidraw/tests/data/__snapshots__/restore.test.ts.snap b/packages/excalidraw/tests/data/__snapshots__/restore.test.ts.snap
@@ -352,6 +352,7 @@ exports[`restoreElements > should restore text element correctly with unknown fo
   "containerId": null,
   "customData": undefined,
   "fillStyle": "solid",
+  "font": "10 unknown",
   "fontFamily": 5,
   "fontSize": 10,
   "frameId": null,