-
Notifications
You must be signed in to change notification settings - Fork 4.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tweak edit site resizable frame handle #50878
Conversation
Size Change: +38 B (0%) Total Size: 1.4 MB
ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. I didn't notice as much of a change in length as your after video, but it's a solid improvement 👍
Nice improvement. For some reason the exit animation doesn't work of the handle (should reverse and slide back in to frame) but that can be treated as a separate issue. |
Yea, I wasn’t too sure about that. Let’s follow up to tweak that bit. |
What?
Vertically centers the new resize handle (#49910), applies standard metrics to the handle, adjusts the hover scaling animations, and ensures the handle stays in its focused state as you're dragging it.
Why?
Makes it feel nicer to interact with.
Testing Instructions
Screenshots or screencast
Before
CleanShot.2023-05-23.at.09.04.31.mp4
After
CleanShot.2023-05-23.at.09.01.08.mp4