fix: Check for roundness undefined in restore.ts

[zsviczian]

null is a valid value for roundness. The current code treats null as "to-be restored".

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
excalidraw	✅ Ready (Inspect)	Visit Preview	May 20, 2024 6:24am
excalidraw-package-example	✅ Ready (Inspect)	Visit Preview	May 20, 2024 6:24am
excalidraw-package-example-with-nextjs	✅ Ready (Inspect)	Visit Preview	May 20, 2024 6:24am

1 Ignored Deployment

Name	Status	Preview	Updated (UTC)
docs	⬜️ Ignored (Inspect)		May 20, 2024 6:24am

[github-actions]

Coverage Report

Status	Category	Percentage	Covered / Total
🔴	Lines	65.56% (🎯 70%)	51254 / 78176
🔴	Statements	65.56% (🎯 70%)	51254 / 78176
🔴	Functions	66.63% (🎯 68%)	1564 / 2347
🟢	Branches	80.99% (🎯 70%)	6294 / 7771

File Coverage

File	Stmts	% Branch	% Funcs	% Lines	Uncovered Lines
Changed Files
packages/excalidraw/data/restore.ts	90.2%	84.56%	78.57%	90.2%	84-87, 150, 200-202, 232-236, 357, 387-397, 442-443, 452-460, 467-476, 485-494, 512-517

Generated in workflow #2535

[dwelle]

null is a valid value for roundness. The current code treats null as "to-be restored".

It doesn't. null is a a fallback:

What issue are you facing?

[zsviczian]

the issue is restoring old drawings in Obsidian. I think the root cause is probably that in Obsidian when I added an image from the Vault it created the Image element with the strokeSharpness property set. In the past this had no effect, but now, when I open old drawings pictures load with round corners.

I am just going to keep this change in my fork and close this PR.