Skip to content

Docs: Modernize video sequence guide - #9100

Merged
JonnyBurger merged 4 commits into
mainfrom
codex/docs-rewrite-video-sequence
Jul 16, 2026
Merged

Docs: Modernize video sequence guide#9100
JonnyBurger merged 4 commits into
mainfrom
codex/docs-rewrite-video-sequence

Conversation

@JonnyBurger

Copy link
Copy Markdown
Member

Summary

  • rewrite the playing-videos-in-sequence guide into three focused steps
  • use <Video> from @remotion/media and include premounting in the first example
  • replace parseMedia() with Mediabunny's Input.computeDuration() and document CORS requirements

Testing

  • bun prewarm-twoslash.ts (from packages/docs)
  • bun run build
  • bun run stylecheck

@vercel

vercel Bot commented Jul 16, 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 16, 2026 9:34am
remotion Ready Ready Preview, Comment Jul 16, 2026 9:34am

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 — modernized the "Playing videos in sequence" how-to guide by replacing <OffthreadVideo> with <Video> from @remotion/media and switching duration discovery from @remotion/media-parser to Mediabunny's Input.computeDuration().

  • Restructured the guide into three clear steps — creating the <Series> of <Video> clips, calculating durations with Mediabunny, and registering the <Composition>.
  • Adopted <Video> and premountFor in the first example — the new component is the documented preferred video component; premountFor replaces the old standalone premounting section.
  • Replaced parseMedia() with Mediabunny — the calculateMetadata() example now uses Input + UrlSource with computeDuration(), matching the canonical patterns in the Mediabunny metadata and <OffthreadVideo> looping docs.
  • Added CORS and native-buffering notes — the rewrite accurately calls out that Mediabunny loads via fetch() and that <Video> uses the Player buffer state by default.

Pullfrog  | View workflow run | Using Kimi K2 (free via Pullfrog for OSS) | 𝕏

@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 in the incremental changes.

Reviewed changes — two follow-up commits adjusting the step markup and reducing the premountFor duration.

  • Fixed <Step> formatting — removed bullet markers and joined the steps with <br />, matching the fragile <Step> component conventions.
  • Reduced premounting from 4 seconds to 1 second — changed premountFor={4 * fps} to {fps} and updated the explanation to say "one second".

Pullfrog  | View workflow run | Using Kimi K2 (free via Pullfrog for OSS) | 𝕏

@JonnyBurger
JonnyBurger merged commit 5e0da59 into main Jul 16, 2026
16 checks passed
@JonnyBurger
JonnyBurger deleted the codex/docs-rewrite-video-sequence branch July 16, 2026 09:37
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