Ignore "use-nesting"

[martijndevalk]

When using the setting "codeActionsOnSave" in VS Code in combination with data-context-theme "@nest" automatically applies which is against our own linting rules.

Is it possible to add this rule so that it prevents this problem as described above?

"rules": { "csstools/use-nesting": "ignore" }

[EECOLOR]

Do you have an example where it causes problems? If I recall correctly we require nesting as well. The current rule is this:

    "csstools/use-nesting": ["always", {
      "except": ["*:checked +"]
    }],

So we mightneed to adjust the rule for a specific use case.

[EECOLOR]

@martijndevalk Could you check this?

[martijndevalk]

screencast.2021-07-06.12-51-49.mp4

[EECOLOR]

@martijndevalk

Wow, I did not know that construction was even possible.

Does the same thing happen when you write the following?

[data-context-theme='jumbo'] {
  & .VacatureDetailBijbaanHero-mediaContainer {
    ...
  }
}

[martijndevalk]

[data-context-theme='jumbo'] {
  & .VacatureDetailBijbaanHero-mediaContainer {
    ...
  }
}

just looked and no then @nest does not seem to be applied

[martijndevalk]

I have another example, now in Alliander.

screencast.2021-08-30.10-58-20.mp4

[EECOLOR]

@martijndevalk I would expect the rule to behave like this:

.transition {
  & > .list {
    margin: 0;
  }
}

Could you ask https://github.com/csstools/stylelint-use-nesting to make the autofix to @nest configurable? We would like autofix for the regular nesting, but not for @nest. If there is no response or no interest to change it we should fork the repository and add the configuration ourselves.

[EECOLOR]

The ticket was create quite some time ago: csstools/stylelint-use-nesting#10

There was no reaction. The last change to the repository was 2 years ago. Next steps:

• Disable autofix for the stylelint-use-nesting plugin in https://github.com/kaliberjs/build/blob/master/library/.stylelintrc
• Move the rules to our own stylelint plugin

@martijndevalk Can jou add the the configuration?