Variable not expanded in url()

[saper]

Example code from sass/node-sass#1042 reported by @chenxiaochun

@mixin ico-common($imgUrl){
    display: inline-block;
    background: url(i/$imgUrl);
    background-repeat: no-repeat;
}

@mixin ico-size($width,$height){
    width: $width;
    height: $height;
}

.test{
    @include ico-common("icon.png");

    @include ico-size(100px, 100px);
}

Ruby Sass output (sass/sass@b4bbd96):

.test {
  display: inline-block;
  background: url(i/"icon.png");
  background-repeat: no-repeat;
  width: 100px;
  height: 100px; }

libsass output as of 3.2.5-111-g5e94 (saper@5e9437b):

.test {
  display: inline-block;
  background: url(i/$imgUrl);
  background-repeat: no-repeat;
  width: 100px;
  height: 100px; }

not sure how useful is url(i/"something.ico") output, but that is what Ruby is doing.

If the Ruby Sass behaviour is undesired, please file an issue at https://github.com/sass/sass/issues

[chenxiaochun]

I am sorry it's my mistake.
It should't has the double quotation mark. But removed, the Ruby Sass will throw exception.

[saper]

so, can we close this?

[chenxiaochun]

OK, thank you very much.

[saper]

@xzyfer I wonder if this expansion issue is a duplicate of some other issue we already have? Or this a new thing and should be kept open?

[xzyfer]

Our url() has some big issues. I'm not sure how defined the behaviour exhibited above is but it's probably worth tracking. I'm not aware of a duplicate to this issue.

[mgreter]

With #1477 it renders now:

.test {
  display: inline-block;
  background: url(i/icon.png);
  background-repeat: no-repeat;
  width: 100px;
  height: 100px; }

[xzyfer]

#1477 addresses the failing interpolation but incorrectly produces an unquote string. This will be addressed in the near future.

[xzyfer]

The incorrect quoting is being tracked in #1480