Skip to content

fix(editor): timeline inside the left column per 346:9830 - #74

Merged
thisiscsim merged 1 commit into
mainfrom
fix/editor-layout-v15
Aug 7, 2026
Merged

thisiscsim merged 1 commit into
mainfrom
fix/editor-layout-v15

Conversation

@thisiscsim

Copy link
Copy Markdown
Owner

Summary

Layout corrections against the Generated the first cut frame (346:9830) — first review with the frame actually rendered (the old View seat's rate cap hit before I could screenshot it during the shell PR).

  • Structure: stage + timeline now form a left column; the right panel runs full height beside them and the timeline ends where the panel starts (it previously spanned the full window width). Timeline resizer moves inside the column; focus mode (Cmd+\) still collapses to just the stage.
  • Stage: preview card gets the layered --ap-shadow-sm from the frame; the panel takes the separating hairline (border-left) instead of the stage.
  • Transport: the left buttons are undo/redo per the mock's curved-arrow glyphs (wired to undoEdl/redoEdl with disabled states — they'd been keyboard-only since the header dropped them); volume/play/time/zoom unchanged.
  • Header: primary Export (share icon) + secondary Settings, matching the frame.
  • Text chips: rosé action tint per the mock (the file styles text chips with the audio action pair, not V0's amber).

Renderer-only — HMR covers it.

Test plan

  • typecheck / test (146) / build / lint / format — green
  • Visual pass against the frame: column split, full-height panel, transport undo/redo, Export button, pink text chips

Made with Cursor

…eft column

Pre-alpha feedback against frame 346:9830: the timeline was spanning the
full window width below both the stage and the panel, but the mock nests
it inside the left column so it ends where the full-height Create panel
begins. App.tsx now wraps stage + timeline in an editor-left column with
the panel beside them. Along the way, the details the rendered frame
exposed: the preview card picks up the layered ap shadow, the panel takes
the separating hairline (stage loses its border), the transport's left
buttons are undo/redo (wired with disabled states; the header dropped
them in the redesign), the header action is a primary Export button with
a secondary Settings, and text chips wear the mock's rosé action tint
instead of the V0 amber.

Co-authored-by: Cursor <cursoragent@cursor.com>
@thisiscsim
thisiscsim merged commit fce58c5 into main Aug 7, 2026
1 check passed
@thisiscsim
thisiscsim deleted the fix/editor-layout-v15 branch August 7, 2026 18:57
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.

1 participant