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

Added: selector-max-universal rule. #2653

Merged
merged 1 commit into from
Jun 26, 2017
Merged

Added: selector-max-universal rule. #2653

merged 1 commit into from
Jun 26, 2017

Conversation

alexander-akait
Copy link
Member

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

#2528

Is there anything in the PR that needs further explanation?

No, it's self explanatory.

Copy link
Member

@hudochenkov hudochenkov left a comment

Choose a reason for hiding this comment

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

Good job!

Please, remove package.json from PR.

column: 1,
}, {
code: "*, \n* * * {}",
description: "multiple selectors: greater than max classes",
Copy link
Member

Choose a reason for hiding this comment

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

"multiple selectors: greater than max universal selectors"

Copy link
Member

@jeddy3 jeddy3 left a comment

Choose a reason for hiding this comment

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

@evilebottnawi Thanks!


```css
/* each selector in a selector list is evaluated separately */
*.foo *,
Copy link
Member

Choose a reason for hiding this comment

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

Please change to:

*.foo,
*.bar * {}

That that only three (one more than the max) universal selectors are used.

```

```css
/* `*` is inside `:not()`, so it is evaluated separately */
Copy link
Member

Choose a reason for hiding this comment

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

Please change to:

* > * .foo:not(*) {}

@jeddy3
Copy link
Member

jeddy3 commented Jun 26, 2017

@hudochenkov I've made the changes if you'd like to look them over?

@hudochenkov
Copy link
Member

@jeddy3 looks good. Thanks!

@jeddy3 jeddy3 force-pushed the selector-max-universal branch from f7ccdda to ebecce3 Compare June 26, 2017 09:05
@jeddy3 jeddy3 force-pushed the selector-max-universal branch from ebecce3 to 8aa12c0 Compare June 26, 2017 09:41
@jeddy3 jeddy3 merged commit ab61481 into master Jun 26, 2017
@jeddy3 jeddy3 deleted the selector-max-universal branch June 26, 2017 10:09
@jeddy3
Copy link
Member

jeddy3 commented Jun 26, 2017

  • Added: selector-max-universal rule (#2653).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants