Skip to content

fix: sync video preview with recipe updates#711

Open
AashitaDash wants to merge 1 commit into
magic-peach:mainfrom
AashitaDash:fix-preview-sync-bug
Open

fix: sync video preview with recipe updates#711
AashitaDash wants to merge 1 commit into
magic-peach:mainfrom
AashitaDash:fix-preview-sync-bug

Conversation

@AashitaDash
Copy link
Copy Markdown

@AashitaDash AashitaDash commented May 19, 2026

Description

This PR fixes a preview synchronization issue in the video editor where preview overlays and recipe-based updates were not reflected immediately after changing editor settings.

Changes Made
Passed recipe prop into VideoPreview
Improved synchronization between editor state and preview rendering
Ensured overlay/framing updates react instantly to recipe changes
Issue Fixed

Previously, VideoPreview was not receiving updated recipe state from VideoEditor, causing stale preview behavior and delayed UI updates.

Type of Change
Bug Fix 🐛
Testing
Tested preset switching
Tested framing overlay updates
Tested live preview synchronization
Verified application builds and runs successfully

Closes #709

@vercel
Copy link
Copy Markdown

vercel Bot commented May 19, 2026

@AashitaDash is attempting to deploy a commit to the magic-peach1's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added level:advanced Advanced level - 55 pts type:bug Bug fix type:design UI/UX design labels May 19, 2026
@github-actions
Copy link
Copy Markdown
Contributor

👋 Thanks for your PR, @AashitaDash!

Welcome to Reframe — a browser-based video editor built for everyone 🎬

What happens next

  1. 🤖 Automated checks — build & TypeScript typecheck will run automatically
  2. Vercel preview — a preview deployment will be created (requires maintainer authorization for fork PRs)
  3. 👀 Code review — a maintainer will review your changes
  4. 🚀 Merge — once approved, your PR will be merged!

Quick checklist

  • PR title follows Conventional Commits (e.g. feat: add dark mode)
  • Linked the issue this PR closes (e.g. Closes #123)
  • Tested the changes locally (bun run dev)
  • Build passes (bun run build)

Useful links

Happy coding! 🎉

@github-actions github-actions Bot added the type:testing Testing label May 19, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 19, 2026

✅ PR Format Check Passed — @AashitaDash

Basic format checks passed. A maintainer will review your code changes.

This does not mean the PR is approved — it just means the format is correct.

@AashitaDash
Copy link
Copy Markdown
Author

Implemented the requested fix and linked the issue. Please review when possible. 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

level:advanced Advanced level - 55 pts type:bug Bug fix type:design UI/UX design type:testing Testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Edited Content Does Not Sync Immediately Across Preview Components

1 participant