Skip to content

fix(editor): span the timeline minimap across the full ruler width - #2958

Open
Ectropy wants to merge 1 commit into
CapSoftware:mainfrom
Ectropy:fix/editor-timeline-minimap-width
Open

Ectropy wants to merge 1 commit into
CapSoftware:mainfrom
Ectropy:fix/editor-timeline-minimap-width

Conversation

@Ectropy

@Ectropy Ectropy commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

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.

image

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.
  • Added the_minimap_spans_the_ruler. The existing minimap drag tests pass.
  • GPUI only. The legacy Tauri editor is unchanged.
  • Tested on Windows 11 Pro in a dev build: dragging the chip, resizing it from both edges, and clicking the empty track to center the view on that point.

Fixes #2874.

Written with Opus 5.5 High. Reviewed and tested by me.

RetriggerView in GreptileConfidence 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.

  • The GPUI minimap now spans the full ruler beneath the tracks.

Reviews (5) · Last reviewed commit: "fix(editor): span the timeline minimap a..." · Reviewed by Greptile

@vercel

vercel Bot commented Oct 23, 2026

Copy link
Copy Markdown
Contributor

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

@Ectropy
Ectropy force-pushed the fix/editor-timeline-minimap-width branch from 68ca8ef to 3056c82 Compare October 23, 2026 14:33
@vercel

vercel Bot commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
cap-status Skipped Skipped Nov 4, 2026 5:38pm UTC

@vercel
vercel Bot temporarily deployed to Preview – cap-status November 2, 2026 02:27 Inactive
@richiemcilroy

Copy link
Copy Markdown
Member

Thanks! Can you get Greptile to 5/5 on the latest commit and get CI green? Then we'll take a look.

@superagent-security superagent-security Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Superagent found 1 security concern(s).

Comment thread apps/web/lib/agent-markdown.ts Resolved
@Ectropy
Ectropy force-pushed the fix/editor-timeline-minimap-width branch from af4f99f to 37f800e Compare November 3, 2026 01:17
@vercel
vercel Bot temporarily deployed to Preview – cap-status November 3, 2026 01:17 Inactive
@Ectropy

Ectropy commented Nov 3, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai review

@Ectropy

Ectropy commented Nov 3, 2026

Copy link
Copy Markdown
Contributor Author

@richiemcilroy
As green as it can get, I think. The failing "Linux desktop reliability / Record on Hyprland (pull_request)" seems to be a pre-existing CI failure that other recent PRs are hitting.

I've tested the Editor personally, and it appears to still behave as expected

@Ectropy
Ectropy force-pushed the fix/editor-timeline-minimap-width branch from 37f800e to addfe44 Compare November 3, 2026 03:41
@vercel
vercel Bot temporarily deployed to Preview – cap-status November 3, 2026 03:41 Inactive
@Ectropy

Ectropy commented Nov 3, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai review

@Ectropy
Ectropy force-pushed the fix/editor-timeline-minimap-width branch from addfe44 to 1d4cf9b Compare November 3, 2026 23:58
@vercel
vercel Bot temporarily deployed to Preview – cap-status November 3, 2026 23:58 Inactive
@Ectropy

Ectropy commented Nov 4, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai review

@Ectropy
Ectropy force-pushed the fix/editor-timeline-minimap-width branch from 1d4cf9b to 42dcffb Compare November 4, 2026 05:42
@vercel
vercel Bot temporarily deployed to Preview – cap-status November 4, 2026 05:42 Inactive
@richiemcilroy

Copy link
Copy Markdown
Member

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
@Ectropy
Ectropy force-pushed the fix/editor-timeline-minimap-width branch from 42dcffb to 0d59024 Compare November 4, 2026 17:38
@Ectropy

Ectropy commented Nov 4, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai review

@vercel
vercel Bot temporarily deployed to Preview – cap-status November 4, 2026 17:38 Inactive
@Ectropy

Ectropy commented Nov 4, 2026

Copy link
Copy Markdown
Contributor Author

@richiemcilroy Greptile 5/5 on latest commit. Ready for your look

This branch was previously deployed

1 inactive deployment
Preview – cap-status — 0d590242 Deployed Nov 4, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Editor's horizontal scroll bar is smaller than expected

2 participants