Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduser bundle-størrelse for Framer Motion #4116

Merged
merged 6 commits into from
Sep 25, 2024
Merged

Conversation

piofinn
Copy link
Contributor

@piofinn piofinn commented Sep 23, 2024

Bruk lazy innlasting, og en modulær versjon av motion-biblioteket fra Framer Motion for å redusere bundle-størrelse i komponentne som bruker dette. Tar utangspunkt i guiden på Framer sine sider.

En rask test med et tomt Vite-prosjekt ga ca. 40 kB besparelse i ferdig bundle etter endringene (før gzip)!

Legger også til støtte for flere versjoner av Framer Motion, for å unngå konflikter dersom konsumenten allerede bruker Framer Motion andre steder. Har ikke testet alle versjoner, men har testet med både laveste og høyeste tillatte versjon og det fungerer fint. Det er ingen breaking changes i API-ene som brukes mellom disse versjonene.

🎯 Sjekkliste

@fremtind-bot
Copy link
Collaborator

fremtind-bot commented Sep 23, 2024

Forhåndsvisning: https://jokul.fremtind.no/preview/feat/lighter-framer-motion/
🔍 Commit: b86a145

Forhåndsvisningen blir tilgjengelig innen et par minutter. Den fjernes automatisk når pull requesten lukkes.

fremtind-bot added a commit that referenced this pull request Sep 23, 2024
fremtind-bot added a commit that referenced this pull request Sep 23, 2024
ivarni
ivarni previously approved these changes Sep 24, 2024
@ivarni
Copy link
Contributor

ivarni commented Sep 24, 2024

Mistenker at testen som feiler kanskje må vente på async lasting av framer-motion eller noe lignende, uten at jeg har undersøkt.

kristianulv23
kristianulv23 previously approved these changes Sep 24, 2024
@piofinn
Copy link
Contributor Author

piofinn commented Sep 24, 2024

Mistenker at testen som feiler kanskje må vente på async lasting av framer-motion eller noe lignende, uten at jeg har undersøkt.

Jepp, går også ut fra det! Rart at den ikke feilet før, men kan hende det er noen timing-forskjeller mellom Framer-versjonene. Skal putte det i en act så får vi satse på at det gjør susen!

@piofinn piofinn dismissed stale reviews from kristianulv23 and ivarni via cf1d8ea September 24, 2024 08:20
fremtind-bot added a commit that referenced this pull request Sep 24, 2024
@piofinn piofinn force-pushed the feat/lighter-framer-motion branch 2 times, most recently from 252d6d0 to adcd2b6 Compare September 24, 2024 20:02
fremtind-bot added a commit that referenced this pull request Sep 24, 2024
ivarni
ivarni previously approved these changes Sep 25, 2024
fremtind-bot added a commit that referenced this pull request Sep 25, 2024
@piofinn piofinn merged commit 3d3bc85 into main Sep 25, 2024
5 checks passed
@piofinn piofinn deleted the feat/lighter-framer-motion branch September 25, 2024 10:42
github-actions bot pushed a commit that referenced this pull request Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants