@remotion/studio: Add timeline duplicate shortcut#7852
Merged
JonnyBurger merged 2 commits intoMay 31, 2026
Merged
Conversation
![pullfrog[bot]](https://avatars.githubusercontent.com/in/1768019?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes — Added a Cmd/Ctrl+D keyboard shortcut for duplicating selected timeline sequence rows, mirroring the existing Backspace delete shortcut and reusing the same /api/duplicate-jsx-node API and notification patterns.
- Register
Cmd/Ctrl+Dkeybinding inTimelineDeleteKeybindings.tsxthat callsduplicateSelectedTimelineItemswith the current selection. - Add
duplicate-selected-timeline-item.tswith confirmation dialogs for programmatically duplicated sequences, type guardisDuplicatableSequenceRowSelectionto restrict duplication to top-level sequence rows, and parallel API calls to duplicate each selected sequence. - Add test coverage in
timeline-selection.test.tsverifying that only sequence rows (not keyframes or effect rows) are considered duplicatable.
Kimi K2 ｜ 𝕏
…rtcut The new Cmd/Ctrl+D shortcut had its own copy of the confirm + duplicate + notification flow that already existed inline in TimelineListItem. Route the context-menu Duplicate action through duplicateSequencesFromSource() so both entry points share the same implementation. Also fix the all-or-nothing cancel: when a multi-selection mixes regular and programmatically duplicated sequences, declining the confirm dialog now only skips the duplicated subset instead of aborting the entire batch. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
Why it changed
This adds the requested keyboard path for duplicating a sequence from the timeline, matching the existing source-level duplicate action.
Fixes #7849
Tests run
bun test packages/studio/src/test/timeline-selection.test.tsbun run formattinginpackages/studiobun run lint -- src/components/Timeline/TimelineDeleteKeybindings.tsx src/components/Timeline/duplicate-selected-timeline-item.ts src/test/timeline-selection.test.tsinpackages/studio(passes with existing warnings elsewhere insrc)bun run makeinpackages/studio