Skip to content

@remotion/studio: Don't show empty layer when there are no sequences#6932

Merged
JonnyBurger merged 1 commit intomainfrom
feature/hide-empty-timeline-layer
Mar 30, 2026
Merged

@remotion/studio: Don't show empty layer when there are no sequences#6932
JonnyBurger merged 1 commit intomainfrom
feature/hide-empty-timeline-layer

Conversation

@JonnyBurger
Copy link
Copy Markdown
Member

Summary

  • Remove the placeholder layer that was shown when no sequences exist
  • calculateTimeline() now returns [] instead of a synthetic empty sequence
  • Remove the now-unused sequenceDuration parameter from calculateTimeline()
  • Update tests to match new behavior

Test plan

  • Open a composition with no sequences — timeline should be empty (no phantom layer)
  • Open a composition with sequences — timeline renders normally

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@pullfrog
Copy link
Copy Markdown
Contributor

pullfrog bot commented Mar 30, 2026

Reviewed PR #6932 — no actionable issues found. The removal of the synthetic empty-sequence placeholder is clean: sequenceDuration parameter fully removed from calculateTimeline, no stale references remain, videoConfigIsNull extraction correctly narrows the useMemo dependency, and tests are updated consistently.

Task list (4/4 completed)
  • Read the PR diff
  • Trace data flow and verify correctness of changes
  • Check for stale references to removed parameter/code
  • Submit review or report progress
Pullfrog  | [View workflow run](https://github.com/remotion-dev/remotion/actions/runs/23750906736/job/69192392995) | Triggered by [Pullfrog](https://pullfrog.com) | Using `Big Pickle` (free) | [𝕏](https://x.com/pullfrogai)

@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Mar 30, 2026

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

Project Deployment Actions Updated (UTC)
bugs Ready Ready Preview, Comment Mar 30, 2026 2:48pm
remotion Ready Ready Preview, Comment Mar 30, 2026 2:48pm

Request Review

@JonnyBurger JonnyBurger merged commit 499c4bf into main Mar 30, 2026
30 of 31 checks passed
@JonnyBurger JonnyBurger deleted the feature/hide-empty-timeline-layer branch March 30, 2026 15:10
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