Skip to content

feat(studio): add timeline keyboard controls#2651

Draft
miguel-heygen wants to merge 1 commit into
codex/studio-timeline-roving-focusfrom
codex/studio-timeline-keyboard-controls
Draft

feat(studio): add timeline keyboard controls#2651
miguel-heygen wants to merge 1 commit into
codex/studio-timeline-roving-focusfrom
codex/studio-timeline-keyboard-controls

Conversation

@miguel-heygen

@miguel-heygen miguel-heygen commented Jul 19, 2026

Copy link
Copy Markdown
Collaborator

What

add timeline keyboard controls.

Why

Keep the virtualized timeline understandable and fully operable with keyboard and assistive technology.

How

Adds the focused selection, treegrid, roving-focus, or keyboard behavior in this stack layer.

Key files in this layer:

  • packages/studio/src/player/components/LayerDisclosureRow.tsx
  • packages/studio/src/player/components/Timeline.tsx
  • packages/studio/src/player/components/Timeline.virtualization.test.tsx
  • packages/studio/src/player/components/TimelineClip.test.tsx

Test plan

Validated on the complete stacked tip with formatting, lint, Studio typecheck, the full production build, and the Studio test suite (2,946 passing; 18 todo). Focused timeline/thumbnail tests and browser QA cover the affected interaction path.

  • Unit tests added/updated
  • Manual testing performed
  • Documentation updated (not applicable for this implementation layer)

Post-Deploy Monitoring & Validation

  • Owner: Studio team
  • Window: First 24 hours after rollout
  • Signals: Watch keyboard navigation, focus restoration, accessibility checks, and inspector-selection consistency.
  • Rollback trigger: Reproducible data loss, broken timeline editing/navigation, or sustained performance regression; revert this stack layer and its descendants.

This was referenced Jul 19, 2026

Copy link
Copy Markdown
Collaborator Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

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