Skip to content

Cue List View: per-pane visibility toggle persisted in URL hash#9

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/rework-cue-list-view
Draft

Cue List View: per-pane visibility toggle persisted in URL hash#9
Copilot wants to merge 2 commits intomainfrom
copilot/rework-cue-list-view

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 4, 2026

The Cue List View was always split into two panes with no way to focus on just one. This adds a three-way toggle — Top only / Both / Bottom only — with the selection round-tripped through the URL hash so it survives page loads and can be shared via QR code.

Store

  • Added CueListViewMode = "both" | "top" | "bottom" type and viewMode field to CueListViewState (default: "both")

Hash persistence

  • parseHash now reads view=top|bottom|both alongside trackId / identifier
  • updateHash writes view back; omitted entirely when "both" to keep default URLs clean

Layout

  • Top pane, splitter, and bottom pane are conditionally rendered based on viewMode
  • Single-pane mode uses flex-1 to fill available height instead of a fixed percentage
  • Splitter drag logic unchanged and only mounted when viewMode === "both"

Toggle UI

  • Three icon buttons (PanelTop / Rows2 / PanelBottom) embedded in the bottom-pane controls header
  • When in top-only mode (bottom pane hidden), a thin bar at screen bottom keeps the toggle reachable

Copilot AI changed the title [WIP] Update Cue List View to toggle pane visibility Cue List View: per-pane visibility toggle persisted in URL hash Apr 4, 2026
Copilot AI requested a review from jstarpl April 4, 2026 10:15
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.

2 participants