Marku I didn't realize this is another "Sass parser error", I have one more case with false error. Should I reopen the original issue

Please ignore comment #1, I've submitted by accident.


#2 Formula inside media query

@media only tv and (resolution: 120 * 3 + "dpi"){

}

=> unexpected token star and number

(In reply to comment #1)
> Marku I didn't realize this is another "Sass parser error", I have one more
> case with false error. Should I reopen the original issue
Keep the current state. Use this one for possible upcoming parsing issues. Thanks.

changeset:   258935:f66878b45532
summary:     #233038#c2 - CP expression in media query

I'm afraid comment#0 ("url($a + $b)" issue) is very tricky to fix.

Normally the CSS grammar creates an URI token for the content of the parens: uri(alink). 

In the described case the content of the parens does not conform the set of allowed chars inside the URI.  

However as the lexer already started to consume the input an error token is created for the whole expressin instead of creating IDENT(url), LPAREN('('), SASS_VAR($a), ...

I tried to remove the URI token completely and replace it by 'uri' parser rule, but then I run into problems in the grammar when the content of the uri is a generic non-quoted URL (like: http://nazdar.org/soubor.txt) in such case it looks like I'd have to introduce a complete grammar for the URI content or create some custom parser which handles this.

Opposite approach - extend the URI token to contain anything doesn't work well either. I may rewrite the rule to:

URI: 'url(' ~')'* ')'; 

but this will work only for the case where the content doesn't contain the rparen, like url(add($a,$b)). 

Moreover such solution wouldn't make the SASS features work inside the URI, it would just relax the errors.

So ...  spent almost whole day on that and still have nothing reliable :-) 

As the issue doesn't seem to be very common, I'll either live with that for 7.4 or if I had some more time I'd create the custom parser (predicate) for the URI rule as mentioned above.

Thank you for evaluation. I can't comment on how common it is as I really don't know. I've seen it once.

Integrated into 'main-silver', will be available in build *201307222300* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)

Changeset: http://hg.netbeans.org/main-silver/rev/f66878b45532
User: Marek Fukala <mfukala@netbeans.org>
Log: #233038#c2 - CP expression in media query

*** Bug 233216 has been marked as a duplicate of this bug. ***

To be thorough, another instance of #1:

$path: '/test/';
div{
    background: url($path+'nic');
}

Next: "Unexpected token LBRACE found" after 'before'

#3: 
@mixin m ( $selector, $hiResIcon)
{

    @media only screen and (min-resolution: 1000) {

      #{$selector}:before {
        content: $hiResIcon;
      }
    }
}

fixed Petr's case (#3):

changeset:   259995:616739c71f65
summary:     #233038#c10 - fixed parsing of selector w/ interpolation expression in media body

Integrated into 'main-silver', will be available in build *201308012300* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)

Changeset: http://hg.netbeans.org/main-silver/rev/616739c71f65
User: Marek Fukala <mfukala@netbeans.org>
Log: #233038#c10 - fixed parsing of selector w/ interpolation expression in media body

#4 -- the parent reference in selector:

@mixin block() {
    @if $var {
        &>li {
            width: 100%/$var;
            &:nth-of-type(#{var}n+1) { clear: both }
        }
    }
}

*** Bug 233051 has been marked as a duplicate of this bug. ***

Created attachment 141026 [details]
Unexpected token RPAREN found

Created attachment 141027 [details]
Unexpected token HASH_SYMBOL found

Created attachment 141028 [details]
Unexpected token LESS_AND found

Created attachment 141029 [details]
Unexpected token COMMA found

This four are from Bourbone mixin library (http://bourbon.io/). Reproduced on 7.4 RC2 and 201310120001.

For me the parser breaks on:

&:nth-child(#{$step + 'n+' + $start}) {
	border-left-style: dotted;
}

with multiple reported errors, among many other things.

---

Side note: I downloaded and installed NetBeans 7.4 just for the native SASS support and I've been forced to go back to http://plugins.netbeans.org/plugin/34929/ plugin to have the files highlighted. Furthermore, even with the plugin the .scss file icons in 'Projects' have those "parser errors" exclamation marks, which I have no idea how to get rid of. It'd seem adequate to remove SASS support from declared "new features" in 7.4 until these issues are fixed.

Created attachment 143014 [details]
Variable interpolation with ID hash

Further variable interpolation problems.

Hope to see an update for this soon. Can't use Netbeans for my newer projects without this. Agree with comment 20 that S[A/C]SS support should be dropped from feature list until fix is implemented. Especially given all these perfectly legal structures that are making the parser puke.

Have the same issue.

background-image: url(../images/icons/#{$name}.svg);
Unexpected token COLON found.

Exact same problem of comment #20 under NetBeans 8.0.1

    &:nth-child(#{$nbMenuTab / 2}) {
        border-right: none;
    }

 > Unexpected token LESS_AND found

The only one unresolved issue from this is url($a + $b)

*** This bug has been marked as a duplicate of bug 238988 ***