search

postcss / postcss-bem-linter

A BEM linter for postcss
MIT License
571 stars 35 forks source link

Allow lint rule that contains @media (#134) #140

Closed tardiva closed 5 years ago

tardiva commented 5 years ago

We are trying to start using this plugin with stylelint-bem-selector-bem-pattern for our scss. While testing it we have found some issues. One of them was described in #134.

.page {
  padding: 20px;  

  &.menu_type_main { // the linter will skip this rule
    @media (min-width: $tablet-landscape) {
      display: block;
    }
  }
}

So, we fixed it and at the same time added another improvement. Comments placed inside the rule are now ignored when extracting selectors.

simonsmith commented 5 years ago

Fantastic, thanks!