fix(web): wrap phase rail dots and center the context meter ring - #7243
fix(web): wrap phase rail dots and center the context meter ring#7243nejim123 wants to merge 2 commits into
Conversation
The Agents panel phase rail rendered one dot per member on a single line, so a phase with many agents (56 verify agents) ran past the panel edge. The pill and its dot cluster now shrink and wrap inside the rail width. The context window meter ring sat 2px left of center. The ring SVG was absolutely positioned inside the Button, whose base class applies a negative horizontal margin to every svg; with left/right anchors that margin became a left shift. The SVG is now a flex child so the margins stay symmetric. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
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 Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
There was a problem hiding this comment.
One finding in PhaseRail: the new min-w-0 on the phase chip lets it shrink below its content, but the title span was given shrink-0 with no truncation, so long phase titles now spill outside the chip's border instead of being clipped. See the inline comment.
Posted via Macroscope — UI Consistency
| <span | ||
| className={cn( | ||
| "font-mono text-[.65rem]", | ||
| "shrink-0 font-mono text-[.65rem]", |
There was a problem hiding this comment.
shrink-0 here works against the min-w-0/max-w-full added to the segment and chip: the chip can now shrink below its content width, but the title can't shrink and has no clipping, so a long phase title renders outside the chip's border (and past the section, which has no overflow-hidden) rather than being contained. Truncating the title instead keeps the chip's border box and single-line geometry intact and matches the workflow-name span in this file (min-w-0 truncate, line 403).
| "shrink-0 font-mono text-[.65rem]", | |
| "min-w-0 truncate font-mono text-[.65rem]", |
Posted via Macroscope — UI Consistency
ApprovabilityVerdict: Approved e779d71 Pure CSS/Tailwind class changes for flexbox wrapping and centering. No logic or runtime behavior changes - only visual layout adjustments. The open review comment is a styling suggestion, not a blocking issue. You can customize Macroscope's approvability policy. Learn more. |
Problem
Two small layout bugs in the web client:
Fix
AgentsPanel.tsx(PhaseRail): the phase wrapper and pill getmin-w-0/max-w-full, the chevron and title getshrink-0, and the dot cluster getsflex-wrap. Dots now wrap inside the pill instead of overflowing. One dot per member is preserved.ContextWindowMeter.tsx: the ring SVG wasabsolute inset-0insideButton, whose base class applies[&_svg]:-mx-0.5to every svg. With left/right anchors that negative margin becomes a 2px left shift. The SVG is now a plain flex child of the centered wrapper, so the margins stay symmetric and the ring is centered. No visual size change.Before / after
Phase rail (before):
Phase rail (after):
Context meter (before, right one shows the hover box):
Context meter (after):
Verification
scrollWidth === clientWidth, ring center offsetdx: 0, dy: 0(wasdx: -2).vp lint,vp fmt --check, andtsgo --noEmitonapps/webare clean.Note
Low Risk
Tailwind-only presentation tweaks with no logic, API, or data changes.
Overview
Fixes two web-only layout bugs in the agents panel and chat context meter.
Agents panel (
PhaseRail): Phase segments and pills now usemin-w-0/max-w-full, chevrons and phase titles useshrink-0, and the member status-dot row usesflex-wrap. Large phases (many agents) keep one dot per member but wrap inside the pill instead of overflowing the panel width.Context window meter: The ring SVG is no longer
absolute inset-0insideButton. It sits in the centered flex wrapper soButton’s[&_svg]:-mx-0.5no longer shifts the ring ~2px left; ring size and behavior are unchanged.Reviewed by Cursor Bugbot for commit 56eade9. Bugbot is set up for automated code reviews on this repo. Configure here.