csscomb / csscomb.js

CSS coding style formatter
http://csscomb.com/
MIT License
3.29k stars 457 forks source link

SCSS mixin call with '!important' param error #457

Closed habovh closed 5 years ago

habovh commented 8 years ago

CSSComb fails to comb a fail is the .scss file contains a mixin that uses the CSS !important keyword as a parameter.

See CSSComb error output:

/Users/Jordan/.nvm/versions/node/v4.2.1/bin/csscomb /Users/Jordan/myCoolApp/styles/_components.modal.scss
stack: /Users/Jordan/myCoolApp/styles/_components.modal.scss
Please check the validity of the block starting from line #61

59 |      border: none !important;
61*|     @include border-radius(0 !important);
62 |     @include media(large) {
63 |         height: 90% !important;

Gonzales PE version: 3.0.0-28
Syntax: scss
CSScomb Core version: 3.0.0-3.1

Process finished with exit code 1

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/32449419-scss-mixin-call-with-important-param-error?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github).
jdalton commented 5 years ago

This is fixed in the latest csscomb release 🙌