-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Upgrade framer-motion #61572
Upgrade framer-motion #61572
Conversation
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Size Change: +2.23 kB (+0.13%) Total Size: 1.75 MB
ℹ️ View Unchanged
|
f301f48
to
b2ffb94
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested all the animations that I can think of in our code and everything seems to be working properly.
ab4326f
to
c6e8268
Compare
There are 3 snapshot failures that are reproducible locally. I would really appreciate help getting this landed if someone can investigate those. |
For me these snapshots are not very useful and ideally the test should be more specific. That said, adding |
I updated the snapshots. |
What?
Upgrade framer-motion package.
This is a significant diff, although I see no release notes to speak of on why it's a major (potentially breaking) version change.
Includes #61532
Extracted from #61486
Why?
This package has type conflicts with recent types, these can manifest causing build issues. framer/motion#2587
How?
Upgrade the package to include the fix.
Testing Instructions
I'm not sure where or how this is used.