Skip to content

Add mobile layout mode for timeline editor#376

Open
smcga wants to merge 1 commit intomainfrom
codex/add-mobile-timeline-editor-option
Open

Add mobile layout mode for timeline editor#376
smcga wants to merge 1 commit intomainfrom
codex/add-mobile-timeline-editor-option

Conversation

@smcga
Copy link
Copy Markdown
Owner

@smcga smcga commented Apr 29, 2026

Motivation

  • Provide a touch-friendly, mobile-first editor layout option so the timeline can be edited on phones without changing the existing desktop editor UX.
  • Keep desktop editor behavior unchanged by making mobile editing opt-in and pluggable.
  • Allow quick entry for QA/development via query param or debug overlay control for easier testing on devices.

Description

  • Add optional layoutMode?: "desktop" | "mobile" to EditorInit and a helper getEditorLayoutClass that returns "editor editor--mobile" for mobile mode and "editor" otherwise.
  • Apply getEditorLayoutClass(init.layoutMode) to the editor container and keep the editor HTML generation unchanged so timeline logic and desktop layout are preserved.
  • Add a debug overlay Editor layout selector in index.html and wire ?editorLayout=mobile query param handling and runtime layout switching in src/main.ts that re-initializes the editor with the chosen layout.
  • Add mobile-specific CSS under .editor--mobile to stack workspace/scenes/inspector and improve preview sizing, update README.md to document the new entry points, and add a unit test for the layout-class helper in src/editor/EditorRoot.test.ts.

Testing

  • Ran unit tests for the editor module with npm test -- src/editor/EditorRoot.test.ts, which passed (66 tests, all green).
  • No timeline files were modified, so sacred musical anchors and timeline timings were not affected.

Codex Task

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 29, 2026

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

Project Deployment Actions Updated (UTC)
codexperimental Ready Ready Preview, Comment Apr 29, 2026 11:50am

Request Review

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant