Skip to content

@remotion/studio: Make pasting keyframes easier - #9430

Merged
JonnyBurger merged 2 commits into
mainfrom
codex/easier-keyframe-pasting
Jul 22, 2026
Merged

@remotion/studio: Make pasting keyframes easier#9430
JonnyBurger merged 2 commits into
mainfrom
codex/easier-keyframe-pasting

Conversation

@JonnyBurger

Copy link
Copy Markdown
Member

Summary

  • include the copied property identity in keyframe clipboard data
  • paste keyframes onto the matching property when a Sequence is selected
  • preserve compatibility with clipboard payloads that do not contain property identity

Testing

  • bun test packages/studio-shared/src/test/keyframe-clipboard-data.test.ts packages/studio/src/test/timeline-selection.test.ts
  • bun run build
  • bun run stylecheck

Closes #9392

@vercel

vercel Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
bugs Ready Ready Preview, Comment Jul 22, 2026 8:12am
remotion Ready Ready Preview, Comment Jul 22, 2026 8:12am

Request Review

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No new issues found.

Reviewed changes — PR adds an optional field identity to keyframe clipboard payloads and uses it to paste copied keyframes onto the matching sequence property when a sequence row is selected, while keeping older payloads that omit field valid.

  • KeyframeClipboardField type and validationpackages/studio-shared/src/keyframe-clipboard-data.ts now carries a field discriminator and rejects malformed identities.
  • Clipboard generationpackages/studio/src/components/Timeline/keyframe-clipboard.ts records field.type/fieldKey when copying keyframes.
  • Paste targeting — when exactly one sequence row is selected and the payload identifies a sequence property, the selection is converted to the matching sequence-prop before resolving the paste target.
  • Tests — parser coverage for invalid field identities and a new test for pasting onto a selected sequence row; existing copy expectations updated.

Local verification passed for the files in scope: bun test, bunx turbo run make, formatting, and lint on @remotion/studio-shared and @remotion/studio.

Pullfrog  | View workflow run | Using Kimi K2 (free via Pullfrog for OSS) (Claude Opus not used — the program covers this model; add its provider key to run your pick) | 𝕏

@JonnyBurger
JonnyBurger merged commit 3b0c5c4 into main Jul 22, 2026
18 checks passed
@JonnyBurger
JonnyBurger deleted the codex/easier-keyframe-pasting branch July 22, 2026 08:12
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.

Pasting keyframes can be easier

1 participant