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 false positives for multiple pseudo-class in selector-pseudo-class-no-unknown #2810

Closed
Mottie opened this issue Aug 13, 2017 · 2 comments
Labels
status: wip is being worked on by someone type: bug a problem with a feature or rule

Comments

@Mottie
Copy link

Mottie commented Aug 13, 2017

Describe the issue. Is it a bug or a feature request (new rule, new option, etc.)?

When multiple pseudoclasses (chained) are added to the webkit scrollbar selector, we see a "selector-pseudo-class-no-unknown" error. In this example css:

::-webkit-scrollbar-button:horizontal:decrement {
  background-color: transparent;
}

::-webkit-scrollbar-button:decrement:horizontal {
  background-color: transparent;
}

::-webkit-scrollbar-button:horizontal {
  background-color: transparent;
}

::-webkit-scrollbar-button:decrement {
  background-color: transparent;
}

The first two selectors show the error:

1:38 error Unexpected unknown pseudo-class selector ":decrement" (selector-pseudo-class-no-unknown)
5:37 error Unexpected unknown pseudo-class selector ":horizontal" (selector-pseudo-class-no-unknown)

The standard stylelint configuration is being used, and the error can be seen after entering the above css into the main stylelint demo.

Which version of stylelint are you using?

8.0.0

How are you running stylelint: CLI, PostCSS plugin, Node API?

Reproducible in the online demo

Does your issue relate to non-standard syntax (e.g. SCSS, nesting, etc.)?

No

What did you expect to happen?

No "selector-pseudo-class-no-unknown" error.

What actually happened (e.g. what warnings or errors you are getting)?

"selector-pseudo-class-no-unknown" error.

@alexander-akait
Copy link
Member

@Mottie thanks for issue, seems it is bug

@alexander-akait alexander-akait added status: ready to implement is ready to be worked on by someone type: bug a problem with a feature or rule status: wip is being worked on by someone and removed status: ready to implement is ready to be worked on by someone labels Aug 14, 2017
@jeddy3 jeddy3 changed the title Webkit scrollbar multiple pseudoclass issue False positive for multiple pseudo-class in selector-pseudo-class-no-unknown Aug 21, 2017
@jeddy3 jeddy3 changed the title False positive for multiple pseudo-class in selector-pseudo-class-no-unknown Fix false positives for multiple pseudo-class in selector-pseudo-class-no-unknown Sep 9, 2017
@ntwb
Copy link
Member

ntwb commented Sep 12, 2017

Fixed via 982024d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: wip is being worked on by someone type: bug a problem with a feature or rule
Development

No branches or pull requests

3 participants