search

sasstools / sass-lint

Pure Node.js Sass linting
MIT License
1.77k stars 532 forks source link

Empty line between blocks: Should media-queries be considered as block? #780

Open tzi opened 8 years ago

tzi commented 8 years ago

Hi!

I'm using sass-lint 1.8.2 with empty-line-between-blocks: 2.

I'm surprised that the following code is considered as valid:

.foo {
  content: 'foo';
  @media (min-width: 600px) {
    content: 'bar';
  }
  @include a-mixin-with-content-argument() {
    content: 'baz';
  }
}

Is media-queries (and mixin with content argument) shouldn't be considered as block?

I can try to propose a PR if it is validate by the community :wink:

Cheers, Thomas.

DanPurdy commented 8 years ago

Yeah, this looks like a bug to me. Should definitely be considered a block. Happy to receive a PR to fix this. Thanks! 👍