fix: don't mutate the bounded text if not updated when submitted

[ad1992]

Attempt fix the "undo twice for first time when bounded text properties updated by selecting text explicitly" mentioned in #4537

Previous

Now

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/excalidraw/excalidraw/86BfnGMynX2iq4KZKm6TTj1u6s6b
✅ Preview: https://excalidraw-git-aakansha-undo-excalidraw.vercel.app

[dwelle]

doesn't this assignment make the prevText variable always equivalent to updatedElement.text, removing the need to cache it at all?

[dwelle]

I noticed a regression:

1. Create a sticky note and write line long enough to wrap one the next line. Confirm.

2. Edit again and delete the text, then write a single-line text. Confirm.

3. The text is misaligned.

   

[ad1992]

I noticed a regression:

1. Create a sticky note and write line long enough to wrap one the next line. Confirm.
2. Edit again and delete the text, then write a single-line text. Confirm.
3. The text is misaligned.
   

Good catch!. Updated the implementation to cache originalText and comparing with editor value instead. Added spec for this case. Should be fine now.

[dwelle]

🚀