fix(web): keep turn minimap stable as composer grows - #6414
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 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 |
ApprovabilityVerdict: Approved 8e518f2 This is a minor UI fix that simplifies the minimap's positioning by removing dynamic bottom inset calculation and using a straightforward CSS You can customize Macroscope's approvability policy. Learn more. |
What Changed
Keep the turn minimap anchored to the full timeline viewport when the composer grows.
Why
Typing a multiline prompt increased the composer's bottom inset on the minimap itself. That shortened the minimap's positioning area and shifted the centered turn rail upward. The composer inset now remains limited to timeline scrolling and content clearance.
UI Changes
Before recording
bnefore.mov
After recording
after.mov
Checklist
Implemented with gpt-5.6-sol via the Codex harness in T3 Code.
Note
Low Risk
Small chat UI layout change; scroll inset behavior for the message list is unchanged.
Overview
Stops the turn minimap from shrinking and drifting when the composer gets taller (e.g. multiline prompts).
TimelineMinimapno longer takesbottomInset/contentInsetEndAdjustment. It is positioned withinset-y-0on the timeline viewport instead oftop-0plus an inlinebottomtied to the composer overlay height.LegendListstill usescontentInsetEndAdjustmentfor end scrolling and content clearance—only the minimap rail stays full-height and stable.Reviewed by Cursor Bugbot for commit 8e518f2. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Fix turn minimap to remain stable as the composer grows
The
TimelineMinimapcomponent was shifting position as the composer grew because it received a dynamicbottomInsetthat adjusted its bottom offset. This removes thebottomInsetprop entirely and anchors the minimap to both top and bottom of its container usinginset-y-0instead oftop-0.Macroscope summarized 8e518f2.