fix: ThreadErrorBanner tooltip#3682
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 Simple UI fix that restructures tooltip/alert component nesting to fix tooltip rendering. The change is purely presentational with no logic or behavior changes, affecting only one component. You can customize Macroscope's approvability policy. Learn more. |
What Changed
ThreadErrorBanner was updated so the tooltip wraps only the error text (), not the whole AlertDescription.
Why
ThreadErrorBanner UI was broken as you can see from this issue #3635.
UI Changes
Checklist
Note
Low Risk
Single-component UI markup fix with no auth, data, or API impact.
Overview
Fixes broken ThreadErrorBanner layout by changing how the error tooltip is composed.
AlertDescriptionnow wraps the fullTooltipblock. TheTooltipTriggeruses aspanwithline-clamp-3for the truncated error text instead of renderingAlertDescriptionas the trigger element, so the dismiss action and alert chrome stay outside the tooltip target.Reviewed by Cursor Bugbot for commit 7773a2c. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Fix tooltip structure in
ThreadErrorBannercomponentRestructures the tooltip in ThreadErrorBanner.tsx so that
AlertDescriptionwraps the entireTooltipblock, rather than being used as theTooltipTriggerrendered element. The trigger is now aspanwithline-clamp-3styling.Macroscope summarized 7773a2c.