[fix/feat:ui] Mute previous tool call row#3227
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 Pure CSS styling changes that adjust text color and font weight on a UI button element. No runtime behavior changes beyond visual appearance. You can customize Macroscope's approvability policy. Learn more. |
What changed
The collapsed previous tool call row now uses muted text.
Why
This row is context for hidden activity, not a tool call result. Muted styling makes that distinction clearer without changing layout.
UI changes
Before:
After:
Checklist
Testing
vp check;vp run typecheckpassed locally before splitting these PRs.Note
Mute tool call toggle button styling in chat messages timeline
Adjusts the styling of the overflow toggle button in the
WorkGroupSectioncomponent in MessagesTimeline.tsx. The button now uses muted text by default with a hover color change, the icon container no longer enforces muted color, and the 'Show fewer tool calls' label loses its medium font weight and specific foreground color class.Macroscope summarized a94157d.
Note
Low Risk
CSS-only tweaks to a single button in the chat timeline; no logic or behavior changes.
Overview
Updates the expand/collapse control in
WorkGroupSection(MessagesTimeline.tsx) so the overflow row reads more like secondary context than a tool result.The button now uses muted text by default (
text-muted-foreground/65) with stronger text on hover, instead of inheriting default foreground. The chevron wrapper no longer applies its own muted color (icons still useopacity-70). When expanded, “Show fewer tool calls” dropsfont-mediumandtext-foreground/82so it matches the muted treatment; the collapsed “+N previous tool call(s)” label is unchanged.Reviewed by Cursor Bugbot for commit a94157d. Bugbot is set up for automated code reviews on this repo. Configure here.