Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Nested @extends possible regression #823

Closed
LaurentGoderre opened this issue Jan 7, 2015 · 3 comments
Closed

Nested @extends possible regression #823

LaurentGoderre opened this issue Jan 7, 2015 · 3 comments

Comments

@LaurentGoderre
Copy link
Contributor

This might be related to #592, the following code is compiling fine in libsass 2.1-beta but no in 3.x

%test {
    > {
        .red {
            color: #F00;
        }
    }
}

p {
    @extend %test;

    > {
        a {
            @extend %test;
        }
    }
}

In 2.1-beta I get

p > .red, p > a > .red {
  color: #F00; 
}

and in 3.x I get

p > .red {
  color: #F00; 
}
@torkiljohnsen
Copy link

I am experiencing something similar. I set up a CodePen snippet where you can compare how CodePen compiles and how LibSASS compiles (added the LibSASS compiled source in a comment). Click the eye icon to toggle compiled CSS view:

http://codepen.io/torkil/pen/BydJQr?editors=110

@xzyfer
Copy link
Contributor

xzyfer commented Feb 16, 2015

I've confirmed this issue is present on master.

Spec added sass/sass-spec#260.

@mgreter mgreter added this to the 3.3 milestone Mar 9, 2015
mgreter added a commit to mgreter/libsass that referenced this issue Apr 2, 2015
@mgreter mgreter modified the milestones: 3.2.1, 3.3 Apr 2, 2015
@mgreter mgreter self-assigned this Apr 2, 2015
mgreter added a commit to mgreter/libsass that referenced this issue Apr 3, 2015
mgreter added a commit to mgreter/libsass that referenced this issue Apr 3, 2015
@mgreter mgreter removed their assignment Apr 8, 2015
@xzyfer xzyfer modified the milestones: 3.2.2, 3.3 May 1, 2015
mgreter added a commit to mgreter/libsass that referenced this issue May 12, 2015
@mgreter mgreter self-assigned this May 12, 2015
@mgreter mgreter modified the milestones: 3.2.4, 3.3 May 12, 2015
@LaurentGoderre
Copy link
Contributor Author

/cc @nschonni !!

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants