Fix duplicate native and custom tooltips#2671
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ApprovabilityVerdict: Approved Mechanical replacement of native You can customize Macroscope's approvability policy. Learn more. |
Dismissing prior approval to re-evaluate f32d96a
Dismissing prior approval to re-evaluate d7b4508
What Changed
This PR removes native browser tooltips from in-app UI surfaces that should use T3 Code's tooltip system instead.
It started with the confirmed double-tooltip bugs, then extends that cleanup across the remaining in-scope web UI controls where native
titletooltips were still being used for hover help, status meaning, or truncated text reveals.Updated areas include:
MessagesTimelinework/tool rows, revert action, and changed-file pillsWhy
Some surfaces could show both the browser's native tooltip and T3 Code's custom tooltip at the same time, which looks broken and feels inconsistent.
Other surfaces were still relying on native
titletooltips even though similar UI already used the T3 Code tooltip system. This change standardizes those hover states so the app uses one tooltip model instead of mixing browser tooltips with custom ones.For compact or icon-first UI, the hover meaning is preserved by moving it to T3 Code tooltips rather than removing it.
UI Changes
Before:
After:
Representative examples:
Checklist
Note
Replace native title tooltips with custom Tooltip components across the web UI
titleattributes from interactive elements throughout the app (diff panel, sidebar, chat composer, message timeline, settings, etc.) and replaces them withTooltip/TooltipTrigger/TooltipPopupcomponents to prevent duplicate native and custom tooltips appearing simultaneously.aria-labelto buttons and controls wheretitlepreviously served as the accessible name.markdown-file-link-tooltip-scrolltotooltip-scrollbar-thinin index.css and sets scrollbar width/height to 4px.aria-labelinstead oftitle.Macroscope summarized d7b4508.