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

supports() clause doesn't support declaration-syntax #1967

Closed
zolotov opened this Issue Jan 14, 2016 · 1 comment

Comments

Projects
None yet
2 participants
@zolotov

zolotov commented Jan 14, 2016

supports() clause support, that was introduced in 3.4.20 is not fully compatible with W3C CSS spec.

According to CSS Cascading and Inheritance Level 4 it's allowed to place supports_conditions OR declaration inside supports() clause.

@import [ <url> | <string> ]
        [ supports( [ <supports-condition> | <declaration> ] ) ]?
        <media-query-list>? ;

So the following imports are both valid and equivalent:

@import "mystyle.css" supports(display: flex);
@import "mystyle.css" supports((display: flex));

However Sass compiler doesn't work for the following code:

@import "mystyle.css" supports(display: flex);

And produces the following error:

Error: Invalid CSS after "....css" supports(": expected ")", was "display: flex);"
        on line 1 of test.scss
@chriseppstein

This comment has been minimized.

Show comment
Hide comment
@chriseppstein

chriseppstein Jan 14, 2016

Member

Thanks, we'll get that fixed in the next patch release.

Member

chriseppstein commented Jan 14, 2016

Thanks, we'll get that fixed in the next patch release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment