If comparison inconsistency between ruby and lib sass

[peterwilsoncc]

It's very minor but a valid comparison in rubySass required refactoring for libSass.

rubySass: @if ( & == $sel ) {

libSass: @if ( "&" == $sel ) {

Full mixin follows:

@mixin last-child($sel: null){
    // for use with the miniselectivzr code in the polyfill JS
    // standard file uses :last-child psuedo class
    // IE file uses the u-last-child utility class
    @if ( "&" == $sel ) {
        @if ( true == $is_ie8_file ) {
            &.u-last-child {
                @content;
            }
        } @else {
            &:last-child {
                @content;
            }
        }
    }
    @else {
        @if ( true == $is_ie8_file ) {
            #{$sel}.u-last-child {
                @content;
            }
        } @else {
            #{$sel}:last-child {
                @content;
            }
        }
    }
}

[xzyfer]

Thanks for the report. Could you please provide a minimal snippet of Sass
code that exhibits this problem so we can debug it?
On 9 Oct 2015 8:51 am, "Peter Wilson" notifications@github.com wrote:

It's very minor but a valid comparison in rubySass required refactoring
for libSass.

rubySass: @if ( & == $sel ) {

libSass: @if ( "&" == $sel ) {

Full mixin follows:

@mixin last-child($sel: null){
// for use with the miniselectivzr code in the polyfill JS
// standard file uses :last-child psuedo class
// IE file uses the u-last-child utility class
@if ( "&" == $sel ) {
@if ( true == $is_ie8_file ) {
&.u-last-child {
@content;
}
} @else {
&:last-child {
@content;
}
}
}
@else {
@if ( true == $is_ie8_file ) {
#{$sel}.u-last-child {
@content;
}
} @else {
#{$sel}:last-child {
@content;
}
}
}
}

—
Reply to this email directly or view it on GitHub
#1590.

[saper]

Simple test written sass/sass-spec#536

[saper]

@peterwilsoncc I think that "&" in libsass does not expose the parent selector, it just compares strings. Can you confirm this is your case as well? See the test case and the expected output from Ruby Sass taken from saper/sass-spec@95053d9

[peterwilsoncc]

Thanks @saper, very much appreciated.