-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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
[Skeleton] Apply the wave animation to the correct element #43474
Conversation
Netlify deploy previewhttps://deploy-preview-43474--material-ui.netlify.app/ Bundle size reportDetails of bundle changes (Toolpad) |
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 wonder how this got through Argos 😅
Probably worth it releasing a patch with this one.
It's an animation.
I am keeping an eye on the issues, let's see if there is something else reported in the first 24 hours that we should fix, and decide tomorrow if it's worth doing a release :) |
@mnajdova @DiegoAndai FWIW we are using Chromatic for our visual testing (we don't have Argos but it's interesting alternative). Chromatic indicated difference in several of our stories which were making use of Skeleton component, hence the issue |
Interesting, we'll look into it. We never really tested animations so far (at least not visually). |
Fixes #43470. While migrating to the variants API, we forgot to add the selector for the ::after element when applying the wave animation, check v5 and instead we had it on the root element.
Tested that it works with both Emotion & Pigment CSS.
Broken behavior: https://mui.com/material-ui/react-skeleton/#animations
Fixed behavior: https://deploy-preview-43474--material-ui.netlify.app/material-ui/react-skeleton/#animations