fix(ui): constrain tooltip width and remove question mark cursor#3752
fix(ui): constrain tooltip width and remove question mark cursor#3752waleedlatif1 merged 1 commit intostagingfrom
Conversation
- Add max-w-[260px] to Tooltip.Content so video previews don't blow out the tooltip size - Replace cursor-help with cursor-default on info icons in settings
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview Updates the settings info icons in Written by Cursor Bugbot for commit feac38f. Configure here. |
Greptile SummaryThis is a small, focused UI polish PR that fixes two tooltip-related issues. A Key changes:
Both changes are minimal, well-scoped, and consistent with the existing code style. Confidence Score: 5/5
Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Info icon hover] --> B[Tooltip.Root triggers]
B --> C[Tooltip.Content rendered in Portal]
C --> D{Has Preview?}
D -- Yes --> E[Render text + Tooltip.Preview\n width capped at max-w-260px]
D -- No --> F[Render text only\n width capped at max-w-260px]
E --> G[Video/Image fills full\ntoolip width via w-full\nand neg. margins -mx-8px]
Reviews (1): Last reviewed commit: "fix(ui): constrain tooltip width and rem..." | Re-trigger Greptile |
- Add max-w-[260px] to Tooltip.Content so video previews don't blow out the tooltip size - Replace cursor-help with cursor-default on info icons in settings
Summary
max-w-[260px]toTooltip.Contentso tooltips with video previews stay a consistent size instead of blowing outcursor-helpwithcursor-defaulton info icons in settings to remove the browser question mark cursorType of Change
Testing
Tested manually
Checklist