Skip to content

fix(web): keep turn minimap stable as composer grows - #6414

Open
extoci wants to merge 1 commit into
pingdotgg:mainfrom
extoci:t3code/fix-composer-turn-indicator
Open

fix(web): keep turn minimap stable as composer grows#6414
extoci wants to merge 1 commit into
pingdotgg:mainfrom
extoci:t3code/fix-composer-turn-indicator

Conversation

@extoci

@extoci extoci commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

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

  • This PR is small and focused
  • I explained what changed and why
  • I included a video for animation/interaction changes

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).

TimelineMinimap no longer takes bottomInset / contentInsetEndAdjustment. It is positioned with inset-y-0 on the timeline viewport instead of top-0 plus an inline bottom tied to the composer overlay height. LegendList still uses contentInsetEndAdjustment for 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 TimelineMinimap component was shifting position as the composer grew because it received a dynamic bottomInset that adjusted its bottom offset. This removes the bottomInset prop entirely and anchors the minimap to both top and bottom of its container using inset-y-0 instead of top-0.

Macroscope summarized 8e518f2.

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: bcb69d72-2d0d-4504-a4ea-0a655e838465

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Aug 13, 2026
@macroscopeapp

macroscopeapp Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: 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 inset-y-0 class instead. The change removes complexity and has clear, limited scope.

You can customize Macroscope's approvability policy. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant