-
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
Tooltip: Fix jitter at edge of screen by enabling __unstableShift #41524
Conversation
Size Change: +4 B (0%) Total Size: 1.24 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.
Nice work 👍
This is testing well for me. The approach looks consistent with the block popover's as well.
✅ Could replicate the issue on trunk
✅ This PR does resolve the problem
Before | After |
---|---|
Screen.Recording.2022-06-03.at.3.03.38.pm.mp4 |
Screen.Recording.2022-06-03.at.3.05.08.pm.mp4 |
There's just a minor typo in the changelog, once that's tweaked this should be right to go 🚢
Co-authored-by: Aaron Robertshaw <60436221+aaronrobertshaw@users.noreply.github.com>
What?
Fix an issue with Tooltip where it would sometimes jitter when rendered at the edge of the screen.
Why?
As of #41402 the Popover sets
__unstableShift
asfalse
by default. The Tooltip component requires this flag to render correctly, so let's opt it in.How?
Opt the Tooltip component in to using the
__unstableShift
flag in the Popover.Testing Instructions
Screenshots or screencast