David Luzar
David Luzar
First step would be to try and come up with a better default, likely adaptive (depending on shape size). One of our principles at Excalidraw is reducing as much UI...
We can also use the squircle algo vjeux pointed out https://twitter.com/Vjeux/status/1467894870383607813 — playing with it here https://squircle-svg.com/ looks like it's exactly what we want: stable radius across sizes. What remains...
I can't reproduce myself. Does it reproduce here? https://excalidraw-dq7zpgzuc-excalidraw.vercel.app/
@SkoricIT can you post your Excalidraw version? Right-click on canvas -> Stats for nerds -> click on the version to copy and paste it here. Thanks!
Alright, that's latest. (The code you pasted reproduces for me until to the point I enter the text editor, then it fixes itself.)
Ok, I can reproduce on Firefox 109, but only on MacOS. Though if I copy out the element, it carries over to other browsers / OS until refresh/edit (as mentioned...
@SkoricIT thanks! Now that we can reproduce we should be good :)
> Will make the change so that it gets called only on `elements` or `appState` change. I don't think we should be doing that just yet. It'll slow down the...
Of course it shouldn't. What I'm pointing out that it'll impact perf of: 1. excalidraw.com (unless we do it inside the package). 2. Potentially even the host app, because the...
You can do two things: 1. subscribe to `excalidrawRef.current.readyPromise(callback)` 2. use a `callbackRef` instead: it's what we're doing for excalidraw.com (with a bit of sugar) ↓ https://github.com/excalidraw/excalidraw/blob/9dd2257932f458cc5a873138e51f0c745a79f187/src/excalidraw-app/index.tsx#L168-L171