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

condition validaiotn not working #1318

Open
kumaresan-subramani opened this issue Apr 12, 2021 · 1 comment
Open

condition validaiotn not working #1318

kumaresan-subramani opened this issue Apr 12, 2021 · 1 comment

Comments

@kumaresan-subramani
Copy link

What version of Sass Lint are you using?

"sass-lint": "^1.13.1"

Please include any relevant parts of your configuration

What did you do? Please include the actual source code causing the issue.

$skin-name : 'sds';
 
@if $skin-name != 'bootstrap' {
   .e-de-ctn .e-de-bzr-button:active {
     @if $skin-name != 'bootstrap4' {
       box-shadow: none;
     }
     @if $skin-name == 'fabric' or $skin-name = 'highcontrast' {
       border-radius: 0;
     }
     @if $skin-name != 'material' {
       background-color: 'red';
       border-color: 'red';
       color: 'red';
     }
   }
   @if $skin-name == 'highcontrast' {
     .e-de-ctn .e-de-bzr-button:active .e-btn-icon {
       color: 'red';
     }
   }
 }

What did you expect to happen?

I have try to validate this scss using sass-lint but it wont return any error.

What actually happened? Please include any error messages given to you by Sass Lint.

BUt i can able to reproduce the issue in online sass to CSS converter, please refer below:

https://jsonformatter.org/scss-to-css#save

Stesp taken

Compiler:

Currently we are using the “gulp-sass” compiler to compile scss to css, we have upgraded the compiler to latest version and issue was not reproduced in that too.

We have tried to reproduce reported issue In local but we are not able to reproduce in local.

Steps taken to reproduce issue in locally:

• Used latest gulp-sass-lint
• Used latest sass-lint
• ‘node-sass’ compiler.
• Latest ‘gulp-sass’ compiler

@kumaresan-subramani
Copy link
Author

Issue

 @if $skin-name == 'fabric' or $skin-name = 'highcontrast' {
       border-radius: 0;
     }

Here i am using single = instead of double ==

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

No branches or pull requests

1 participant