libsass throws on non-selector psuedo selector params

[jquense]

input.scss

This is a hacky syntax that css-modules uses to define dependecies across files, in general we'd want sass to ignore it, which is what the other sasses do.

:import("./foo") {
  color: red;
}

Actual results

libsass 3.5.4

Invalid CSS after ":import(": expected selector, was '"./foo") {' on line 1 at column 1

Expected result

This is what dart and ruby sass compile to

:import("./foo") {
  color: red;
}

[nschonni]

@nex3 should Dart be erroring here too?

[jquense]

ruby also allows this. Ideally (for me) libsass would NOT error. I was expecting it to behave more like css parsers and just ignore stuff it doesn't understand. I realize why sass might not do that tho. In our case this is more about being compatible with the very limited pseudo syntax css-modules requires

[nex3]

@nschonni No; unknown pseudo selectors with arguments should allow a <declaration-value> (effectively allowing any contents).

[jquense]

Happy to try and fix this myself. Dug into it a bit and generally identified where to make the change, but can't figure out how to consume the the content, or what it should be stored in, same as css custom property value?

I'll add that i'm a JS dev not a Cpp one, so please excuse me if I'm making some obvious blunder !