Defining a placeholder within grouped selectors

[michaek]

Currently, if we define grouped selectors like:

.a, %a {
  color: red;
}

and extend them like:

.b {
  @extend %a;
}

the output is:

.b {
  color: red;
}

The addition of the placeholder selector causes the entire group of selectors where it is defined to be removed from the output CSS. The expected output would be:

.a, .b {
  color: red;
}

This is a well-known issue, for instance documented by Ben Frain here. I am creating this issue for the purpose of tracking this particular bug, and creating a related spec in sass-spec.

This may be considered a child issue of #146.

[michaek]

This looks like a partial duplicate of https://github.com/sass/sass-spec/tree/master/spec/todo/basic-extend-tests/190_test_semi_placeholder_selector, but that test does not verify output of the use of extend, just the definition of a grouped placeholder selector. Also, it is testing two things: descendant selectors and selector grouping. It may be preferable to have a test that only tests grouping.

[kleinfreund]

Reproduced this over there: dlmanning/gulp-sass#65

Steps to reproduce:

.clearfix:after,
%clearfix:after {
    content: "";
    display: table;
    clear: both;
}

.site-nav {
    @extend %clearfix;
}

Expected output:

.clearfix:after, .site-nav:after {
    content: "";
    display: table;
    clear: both; }

Actual output:

.site-nav:after {
    content: "";
    display: table;
    clear: both; }

I tested if it works with sass sass/main.scss css/test.css and it does.

• gulp-sass v0.7.2
• sass v3.3.9

(Doesn't work with current versions either.)

@michaek

Can you reproduce this comment?

[michaek]

Thanks, @kleinfreund. Your example is reproducible, for example http://sassmeister.com/gist/c9217fcd6806435ab588, but there's also existing coverage for it in the sass-spec suite. (In fact, there appear to be two tests covering it - the second is here https://github.com/sass/sass-spec/blob/master/spec/todo/regressions/placeholder-mediaquery/input.scss !)

[HamptonMakes]

This has been fixed with the merger of the extends branch.