@remotion/studio: Skip mounting volume overlay canvas when volume is default

[JonnyBurger]

Summary

• Skip mounting the volume overlay canvas in AudioWaveform when volume is not dynamic
• Keep overlay rendering behavior unchanged for dynamic string-based volume automation

Testing

• bun run build
• bun run formatting

Closes #7760

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
bugs	Ready	Preview, Comment	May 28, 2026 4:42pm
remotion	Ready	Preview, Comment	May 28, 2026 4:42pm

[pullfrog]

✅ No new issues found.

Reviewed changes — Skip mounting the volume overlay canvas in AudioWaveform when volume is a static number rather than a dynamic string-based automation.

• Introduce shouldRenderVolumeOverlay — Derived from doesVolumeChange && typeof volume === 'string', replacing the inline early-return guard inside the volume overlay useEffect.
• Conditionally render the volume canvas — The <canvas ref={volumeCanvas}> element is now only mounted in the DOM when volume automation is present, avoiding an unnecessary DOM node.
• Update effect dependencies — The volume overlay useEffect dependency array now keys off shouldRenderVolumeOverlay instead of doesVolumeChange, which is equivalent because the boolean is derived from the same inputs.

  ^{｜ View workflow run ｜ Using Kimi K2 ｜ 𝕏}