Conversation
Render tooltip at document root so that it is always visible over other elements. Co-authored-by: roaring30s <okaaycrypto@gmail.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull request overview
This PR fixes tooltip occlusion in the Explorer UI by rendering the desktop tooltip content in a portal at the document root, preventing it from being clipped/covered by other elements on the page (as reported in issue #544).
Changes:
- Wraps the desktop tooltip
ContentinTooltip.Portalso it renders at the document root.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <Tooltip.Portal> | ||
| <Content side="top" align="center" sideOffset={5} multiline {...props}> | ||
| <Text |
There was a problem hiding this comment.
PR description says it adds a "tooltip.root wrapper" to render the tooltip at document root, but the actual change is adding Tooltip.Portal. Consider updating the PR description (or changelog) to reference the Portal to avoid confusion for future readers.
![vercel[bot]](https://avatars.githubusercontent.com/in/8329?s=60&v=4){kind=small}
…dals/drawers despite using Portal Co-authored-by: rickstaa <rick.staa@outlook.com>
|
@rickstaa I like this. This would also prevent future tooltip collisions from happening and put this problem to rest once and for all. |
2 participants
Description
Render tooltip at document root so that it is always visible over other
elements.
Type of Change
Related Issue(s)
Closes: #544
Changes Made
Testing
How to test (optional unless test is not trivial)
Impact / Risk
Risk level: Low / Medium / High
Impacted areas: UI / API / DB / Infra / Config / ...
User impact: Better UI
Rollback plan: Revert
Screenshots / Recordings (if applicable)
Additional Notes