Repository navigation
Conversation
|
@Ectropy is attempting to deploy a commit to the Cap Software Inc Team on Vercel. A member of the Team first needs to authorize it. |
68ca8ef to
3056c82
Compare
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
|
Thanks! Can you get Greptile to 5/5 on the latest commit and get CI green? Then we'll take a look. |
af4f99f to
37f800e
Compare
|
@greptileai review |
|
@richiemcilroy I've tested the Editor personally, and it appears to still behave as expected |
37f800e to
addfe44
Compare
|
@greptileai review |
addfe44 to
1d4cf9b
Compare
|
@greptileai review |
1d4cf9b to
42dcffb
Compare
|
Thanks! Can you get Greptile to 5/5 on the latest commit? Then we'll take a look. |
The minimap was capped at 78px and pinned to the bottom-right corner of the timeline card, which made it hard to see and use as a scroll bar. Size it to the ruler width so it runs from the track gutter to the card's right edge. Fixes CapSoftware#2874
42dcffb to
0d59024
Compare
|
@greptileai review |
|
@richiemcilroy Greptile 5/5 on latest commit. Ready for your look |
The timeline minimap (horizontal scroll bar) was capped at 78px and pinned to the bottom-right of the timeline card. That made it hard to see or use as a scroll bar. This sizes it to the ruler width, so it runs from the track gutter to the card's right edge. It works much like the horizontal scroll bar in DaVinci Resolve, which spans the full width of the area directly below the tracks but doesn't extend beneath the track names.
The 78px corner placement came in with ae19682 (it was full-width along the top when first added in 0c24101). If the small size was intentional, I'm happy to close this.
minimap_bounds now returns the full ruler span. The drawn bar and the click/drag hit area both use it, so they stay aligned.the_minimap_spans_the_ruler. The existing minimap drag tests pass.Fixes #2874.
Written with Opus 5.5 High. Reviewed and tested by me.


Confidence Score: 5/5
This PR appears safe to merge.Summary
Expands the GPUI timeline minimap from its 78px corner placement to the full ruler width.
Reviews (5) · Last reviewed commit: "fix(editor): span the timeline minimap a..." · Reviewed by Greptile