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

Add media-prefers-reduced-motion rule #3357

Closed
wants to merge 1 commit into from

Conversation

@YozhikM
Copy link
Contributor

@YozhikM YozhikM commented Jun 3, 2018

Which issue, if any, is this issue related to?

Issue #3356

Is there anything in the PR that needs further explanation?

No.

@YozhikM YozhikM changed the title feat: Add media-prefers-reduced-motion rule Add media-prefers-reduced-motion rule Jun 3, 2018
@ai
Copy link
Member

@ai ai commented Jun 3, 2018

Also good rule for a11y

@jeddy3
Copy link
Member

@jeddy3 jeddy3 commented Jun 4, 2018

See #3355 (comment) about having this live in a plugin first.

Also good rule for a11y

We can promote this plugin in the documentation e.g.

  • Support for plugins: It's easy to create your own rules and add them to the linter, or make use of plugins written by the community. For example, there are plugins for Scss and a11y.
@alexander-akait
Copy link
Member

@alexander-akait alexander-akait commented Jun 4, 2018

@jeddy3 looks like no one starting a11y plugin, i do not think that this can happen, maybe it is normal put this rules in core, if somebody starts working on a11y plugin, we can move this out of stylelint in 10 version. We should will be responsive to community, plugin system is awesome, but it is doesn't make sense when we have little community 😞

@jeddy3
Copy link
Member

@jeddy3 jeddy3 commented Jun 6, 2018

Ported to a plugin in #3364

@YozhikM Thanks again.

@jeddy3 jeddy3 closed this Jun 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants