Skip to content
New issue

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

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

Already on GitHub? Sign in 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 !!

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

Successfully merging a pull request may close this issue.

4 participants