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

can i disable it for some lines? #16

Closed
arcifius opened this issue Mar 17, 2018 · 3 comments
Closed

can i disable it for some lines? #16

arcifius opened this issue Mar 17, 2018 · 3 comments

Comments

@arcifius
Copy link

I'm trying to do:

// sass-lint:disable-all
:root {
    --background-image: url("nonrelevanturl");
}
// sass-lint:enable-all

But linter still warning and stopping the formatting
Error is: Invalid CSS after "...kground-image: ": expected "}", was "url("..."

@sjova
Copy link
Owner

sjova commented Mar 17, 2018

Hi, @arcifius can you paste me some documentation about this: --background-image?

I'm asking this because that part it seems is not valid.

This is not formatted because of error:

:root {
    --background-image: url("nonrelevanturl");
}

And this is formatted properly:

:root {
    background-image: url("nonrelevanturl");
}

So I need detailed documentation and specification where this is defined: --background-image?

Also, linter and/or disabling format for the specific line(s) is not supported at this moment.

@arcifius
Copy link
Author

It is just simple css variables declaration, eg:

:root {
   --myvar1: red;
   --myvar2: blue;
}

wich works fine... The problem is when I try to assign a url to a variable, it works but the linter keeps saying it is wrong and formatting process is aborted...

@sjova
Copy link
Owner

sjova commented Jun 23, 2018

Sass formatter extension is built on top of sass-convert, and reported the issue is the default behavior of sass-convert tool. Also, we're waiting on sass-convert from Dart implementation. At this moment I can't do anything about this.

@sjova sjova closed this as completed Jun 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants