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

fix(skeleton): disable motion if prefers-reduced-motion is true #3905

Merged
merged 3 commits into from
Sep 4, 2019

Conversation

tw15egan
Copy link
Collaborator

@tw15egan tw15egan commented Sep 4, 2019

Refs #2440
Refs #3897

Improvement that can immediately target skeleton components. Having this disable animations throughout the library will be a larger undertaking

Changelog

New

  • Disables animations on skeleton states if prefers-reduced-motion is set to true

Testing / Reviewing

To enable prefers-reduced-motion, follow these steps then refresh the skeleton page:

Screen Shot 2019-09-04 at 1 39 56 PM

@tw15egan tw15egan requested a review from dakahn September 4, 2019 20:42
@tw15egan tw15egan requested review from shixiedesign and a team September 4, 2019 20:42
@ghost ghost requested review from alisonjoseph and removed request for a team September 4, 2019 20:43
@netlify
Copy link

netlify bot commented Sep 4, 2019

Deploy preview for the-carbon-components ready!

Built with commit 310bd40

https://deploy-preview-3905--the-carbon-components.netlify.com

@netlify
Copy link

netlify bot commented Sep 4, 2019

Deploy preview for carbon-elements ready!

Built with commit 310bd40

https://deploy-preview-3905--carbon-elements.netlify.com

@netlify
Copy link

netlify bot commented Sep 4, 2019

Deploy preview for carbon-components-react ready!

Built with commit 310bd40

https://deploy-preview-3905--carbon-components-react.netlify.com

@netlify
Copy link

netlify bot commented Sep 4, 2019

Deploy preview for the-carbon-components ready!

Built with commit fec4d9c

https://deploy-preview-3905--the-carbon-components.netlify.com

Copy link
Contributor

@shixiedesign shixiedesign left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I followed TJ's instruction and reviewed in Firefox – getting expected still gray squares! 🎉 Thanks for getting this done so fast!

@netlify
Copy link

netlify bot commented Sep 4, 2019

Deploy preview for carbon-components-react ready!

Built with commit fec4d9c

https://deploy-preview-3905--carbon-components-react.netlify.com

@netlify
Copy link

netlify bot commented Sep 4, 2019

Deploy preview for carbon-elements ready!

Built with commit fec4d9c

https://deploy-preview-3905--carbon-elements.netlify.com

Copy link
Contributor

@dakahn dakahn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool! 👍

@shixiedesign shixiedesign merged commit d9aede1 into carbon-design-system:master Sep 4, 2019
@tw15egan tw15egan deleted the reduce-motion branch April 28, 2021 18:15
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.

4 participants