Skip to content

@remotion/studio: Show audio waveform in video timeline layers#6949

Merged
JonnyBurger merged 1 commit intomainfrom
feature/video-layer-audio-waveform
Mar 31, 2026
Merged

@remotion/studio: Show audio waveform in video timeline layers#6949
JonnyBurger merged 1 commit intomainfrom
feature/video-layer-audio-waveform

Conversation

@JonnyBurger
Copy link
Copy Markdown
Member

Summary

  • Video timeline layers now render both the video filmstrip and an audio waveform underneath
  • Video layer height increased from 50px to 75px (50px filmstrip + 25px audio)
  • Audio waveform is correctly offset to exclude premount/postmount regions, so the volume line starts from 0 for videos in TransitionSeries with premount

Test plan

  • Open studio with a composition containing a <Video> tag and verify both filmstrip and audio waveform render
  • Check premount-on-transition-series composition — BigBuckBunny.mp4 volume line should start from 0
  • Verify audio-only sequences still render correctly at 25px height
  • Verify image sequences still render at 50px height

🤖 Generated with Claude Code

Video layers now render both the video filmstrip and an audio waveform
underneath. The video layer height is increased from 50px to 75px to
accommodate both. The audio waveform is correctly positioned to exclude
premount/postmount regions.

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

vercel bot commented Mar 31, 2026

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

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

Request Review

@pullfrog
Copy link
Copy Markdown
Contributor

pullfrog bot commented Mar 31, 2026

Reviewed PR #6949 — no actionable issues found. The height refactor, audio waveform integration, and premount offset logic are all correct and consistent across consumers. Approved.

Task list (4/4 completed)
  • Read the diff and identify all changed files
  • Review each changed file in detail
  • Trace data flow and verify correctness
  • Submit review or report progress

Pullfrog  | View workflow run | Triggered by Pullfrog | Using Big Pickle (free) | 𝕏

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.

Clean, correct change. Height constants, layout math, premount/postmount offset logic, and AudioWaveform prop threading all check out. All consumers of getTimelineLayerHeight consistently pick up the new 75px video height.

Pullfrog  | View workflow run | Using Big Pickle (free) | 𝕏

@JonnyBurger JonnyBurger merged commit a380978 into main Mar 31, 2026
18 of 19 checks passed
@JonnyBurger JonnyBurger deleted the feature/video-layer-audio-waveform branch March 31, 2026 14:34
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