Skip to content

@remotion/studio: Show keyframe controls for keyframed props#7857

Merged
JonnyBurger merged 2 commits into
mainfrom
jonnyburger/keyframe-controls
May 31, 2026
Merged

@remotion/studio: Show keyframe controls for keyframed props#7857
JonnyBurger merged 2 commits into
mainfrom
jonnyburger/keyframe-controls

Conversation

@JonnyBurger
Copy link
Copy Markdown
Member

Summary

  • show timeline keyframe left / diamond / right controls when the field is selected
  • also show those controls when the prop status is already keyframed, even if the row is not selected
  • apply the behavior for both sequence field rows and effect field rows

Fixes #7848

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 31, 2026

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

Project Deployment Actions Updated (UTC)
bugs Ready Ready Preview, Comment May 31, 2026 12:48pm
remotion Ready Ready Preview, Comment May 31, 2026 12:48pm

Request Review

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@pullfrog pullfrog Bot left a comment

Choose a reason for hiding this comment

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

Reviewed changes — a small, focused change that shows timeline keyframe navigation controls (left / diamond / right) for keyframed props even when the field row is not selected, applied consistently across both sequence and effect field rows.

  • Extract shared visibility guardshouldShowTimelineKeyframeControls centralizes the logic for when keyframe controls appear, replacing inline ternaries in both TimelineSequenceFieldRow and TimelineEffectFieldRow.
  • Preserve SELECTION_ENABLED gating — the helper correctly respects the feature-flag so the new behavior only surfaces when timeline selection is enabled.

Pullfrog  | View workflow run | Using Kimi K2𝕏

@JonnyBurger JonnyBurger merged commit 54d0f02 into main May 31, 2026
19 checks passed
@JonnyBurger JonnyBurger deleted the jonnyburger/keyframe-controls branch May 31, 2026 12:58
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.

Show keyframe left / diamond / right controls also if there are already keyframes

1 participant