Skip to content

[lexical-playground]: Draggable block handle gliding effect#8042

Merged
ivailop7 merged 5 commits intofacebook:mainfrom
ivailop7:glide_handle
Dec 25, 2025
Merged

[lexical-playground]: Draggable block handle gliding effect#8042
ivailop7 merged 5 commits intofacebook:mainfrom
ivailop7:glide_handle

Conversation

@ivailop7
Copy link
Copy Markdown
Collaborator

@ivailop7 ivailop7 commented Dec 19, 2025

Add a gliding effect to the draggable block handle and a fade-in and fade-out when the handle is shown/hidden.
To remove the 'fly-in' effect due to the current design where the handle is positioned by default at -10000,-10000, made a minor change to make the handle not visible vs positioned off screen when the mouse cursor is not in the textarea. I'm not sure why this was approach taken in a first place. I'm not aware of any accessibility or performance benefits to have it off screen vs hidden.

Screen.Recording.2025-12-19.at.21.25.23.mov

@vercel
Copy link
Copy Markdown

vercel bot commented Dec 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
lexical Ready Ready Preview, Comment Dec 25, 2025 9:01pm
lexical-playground Ready Ready Preview, Comment Dec 25, 2025 9:01pm

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 19, 2025
@etrepum
Copy link
Copy Markdown
Collaborator

etrepum commented Dec 21, 2025

Not sure what is causing the deployment failures, could be something transient? Not sure how to trigger another build without another commit

@ivailop7
Copy link
Copy Markdown
Collaborator Author

Not sure either, thought we blocked Vercel builds, but I see other PRs are fine. Will wait on the big pnpm PR to go through and then merge these 2.

@zurfyx
Copy link
Copy Markdown
Member

zurfyx commented Dec 22, 2025

Not sure either, thought we blocked Vercel builds, but I see other PRs are fine. Will wait on the big pnpm PR to go through and then merge these 2.

Thank you, pnpm PR is landed. We did block Vercel builds for some time, but at this point in time everything is fully functional.

Copy link
Copy Markdown
Collaborator

@etrepum etrepum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not quite sure what's going on here, maybe some layout race condition, but the handle now gets stuck in offset positions after a drop (until it's moved to another element)

Screen.Recording.2025-12-23.at.10.47.59.mov

@ivailop7
Copy link
Copy Markdown
Collaborator Author

Noted, that's weird. Will have a look in the coming days.

@ivailop7
Copy link
Copy Markdown
Collaborator Author

Fixed the onDrop issue, it was also happening for the very first row when entering the textarea with the cursor. All good now. Merging.

@ivailop7 ivailop7 added this pull request to the merge queue Dec 25, 2025
Merged via the queue into facebook:main with commit 9b6bc2c Dec 25, 2025
35 of 36 checks passed
@etrepum etrepum mentioned this pull request Jan 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. extended-tests Run extended e2e tests on a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants