-
Notifications
You must be signed in to change notification settings - Fork 11.7k
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
VizTooltips: Use global portal #81986
Conversation
Getting an error with this enabled in certain conditions in which it appears to be causing some overflow causing the global scrollbar to flicker. See screen recording: scrollbar.error-2024-02-22_22.34.05.mp4 |
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.
See comment with video 😖
what error are you getting? the occasional flicker close to bottom and right viewport edges is known, but i want to leave the fix for later as it's not a 1-liner - i think we need to render the tooltip initially with |
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.
No worries on the error then, it can be in a follow up PR
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.
⭐
Co-authored-by: Adela Almasan <adela.almasan@grafana.com>
Co-authored-by: Adela Almasan <adela.almasan@grafana.com>
this refactors
newVizTooltips
to use a global portal for tooltips and annotation editor/infotip. this fixes a number of tricky z-index issues with portalling tooltips into the panels, especially with shared tooltips, but also with the panel resize-handle in each panel appearing on top of the tooltips.this also removes the condition to use old tooltips for dashboards with shared tooltip mode. the new behavior is to always use Multi/All mode for tooltips in synced panels.
there will be a follow-up PR to revert previous z-index changes we did in dashboards and scenes that were needed for panel-local portalling:
we could potentially revert #75499, but i think that can stay in place.