perf: implement component and image lazy loading (#1382) - #1386
basantnema31 wants to merge 1 commit into
Conversation
|
@basantnema31 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. |
👋 Thanks for your PR, @basantnema31!Welcome to Reframe — a browser-based video editor built for everyone 🎬 What happens next
Quick checklist
Useful links
Happy coding! 🎉 |
✅ PR Format Check Passed — @basantnema31Basic 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. |
|
Hi! Just a heads up that I'm still actively working on this PR. The implementation is complete — lazy loading has been applied to VideoEditor and all its heavy subcomponents (ExportOverlay, LottiePlayer, etc.) using dynamic imports to improve initial load time and bundle size. The only pending item is the Vercel preview deployment which requires maintainer authorization. Could a maintainer please authorize the Vercel deployment when they get a chance? Happy to make any additional changes based on review feedback. Thank you! 🙏 |
|
Thanks for the contribution! Another open PR (#1564: perf: implement lazy loading and bundle splitting (resolves #1382)) tackles the same issue and is a bit further along (cleaner CI / no merge conflicts), so I'm consolidating there to keep review focused in one place. Closing this one — no reflection on the work itself, just avoiding split review across duplicates. Feel free to bring anything over to #1564 if it adds something that one doesn't have yet! |
Fixes #1382. Implemented dynamic imports for VideoEditor and its heavy subcomponents (ExportOverlay, LottiePlayer, etc) to improve load time and bundle size.