[lexical-playground] Bug Fix: Track Resizing Actions for Excalidraw in History Stack

[neysanfoo]

Description

Current Behaviour

Excalidraw resizing actions are not tracked. My understanding of this is that because the component is fetching width and height from the editor state rather than receiving them as props, React didn't detect the changes to width and height, preventing the component from rerendering on resizing. And as a result, Lexical did not record these transactions in its history stack.

Changes Introduced

I am now passing in the width and height properties as props into ExcalidrawComponent, similar to what is done in ImageNode. And I got rid of useMemo.

Before

excal_resize_before.mov

After

excal_resize_after.mov

Tested in collab mode as well.

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
lexical	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 3, 2024 4:20pm
lexical-playground	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 3, 2024 4:20pm

[github-actions]

size-limit report 📦

Path	Size
lexical - cjs	29.86 KB (0%)
lexical - esm	29.75 KB (0%)
@lexical/rich-text - cjs	38.45 KB (0%)
@lexical/rich-text - esm	31.58 KB (0%)
@lexical/plain-text - cjs	37.09 KB (0%)
@lexical/plain-text - esm	28.95 KB (0%)
@lexical/react - cjs	40.22 KB (0%)
@lexical/react - esm	32.94 KB (0%)