Skip to content

fix: lottie layout shift while lazy loading#4588

Merged
capJavert merged 2 commits intomainfrom
lottie-layout-shift
Jun 13, 2025
Merged

fix: lottie layout shift while lazy loading#4588
capJavert merged 2 commits intomainfrom
lottie-layout-shift

Conversation

@capJavert
Copy link
Contributor

@capJavert capJavert commented Jun 13, 2025

Changes

While lottie JS bundle is loading nothing would be rendered which would cause layout shift once JS loads and animation is shown:

before:

Screen.Recording.2025-06-13.at.14.58.31.mov

after:

Screen.Recording.2025-06-13.at.14.59.36.mov

Events

Did you introduce any new tracking events?

Experiment

Did you introduce any new experiments?

Manual Testing

Caution

Please make sure existing components are not breaking/affected by this PR

Preview domain

https://lottie-layout-shift.preview.app.daily.dev

@vercel
Copy link

vercel bot commented Jun 13, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
daily-webapp ✅ Ready (Inspect) Visit Preview Jun 13, 2025 1:17pm
1 Skipped Deployment
Name Status Preview Updated (UTC)
storybook ⬜️ Ignored (Inspect) Jun 13, 2025 1:17pm

@capJavert capJavert marked this pull request as ready for review June 13, 2025 13:04
@capJavert capJavert requested a review from a team as a code owner June 13, 2025 13:04
@capJavert capJavert enabled auto-merge (squash) June 13, 2025 13:14
@capJavert capJavert merged commit ca2cbf5 into main Jun 13, 2025
10 checks passed
@capJavert capJavert deleted the lottie-layout-shift branch June 13, 2025 13:17
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.

2 participants