-
-
Notifications
You must be signed in to change notification settings - Fork 78.9k
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
Add transition to property-blacklist #26922
Conversation
@MartijnCuppens: there are more instances of transition in docs. |
Changed it. The stylelint check also happens on the custom css files. There are several ways we can handle this:
@media screen and (prefers-reduced-motion: reduce) {
.selector {
transition: none;
}
}
Which way do you want to go with this? |
Hmmm... not sure to be honest. @andresgalante @mdo thoughts? |
@MartijnCuppens: Perhaps you could to the same for other properties? About the examples, we have a .stylelintrc in the examples folder. You can disable the check there instead of each CSS file. https://github.com/twbs/bootstrap/blob/v4-dev/site/docs/4.1/examples/.stylelintrc |
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.
Thanks!
c299670
to
493ec42
Compare
In each PR, we need to check manually if the
@transition
mixin is used instead of thetransition
property. By adding this stylelint rule, thecss-lint
command will take care of this.