Skip to content

Pseudo-class support doesn’t work well with sass @extend (comma-separated selectors) #39

Closed
alanhogan opened this Issue Jun 4, 2012 · 0 comments

2 participants

@alanhogan

My Sass (using Sass 3.2):

%foo
  background-color: red
  &:active
    background-color: pink

.foo-big
  @extend %foo
  font-size: 24px

.foo-little
  @extend %foo
  font-size: 16px

This ends up generating CSS like

.foo-big, .foo-little {
  background-color: red;
}
.foo-big {
  font-size: 24px;
}
.foo-little {
  font-size: 16px;
}

.foo-big:active, .foo-little:active {
  background-color: pink;
}

KSS seems to be generating this CSS:

.foo-big.pseudo-class-active, .foo-little:active {
  background-color: pink;
}

Note only one of those comma-separated selectors was transmuted to its real-class form. As a result, the .foo-little:active KSS preview looks broken.

@alanhogan alanhogan added a commit to alanhogan/kss that referenced this issue Jun 4, 2012
@alanhogan alanhogan Fix kneath/kss#39 3baf6df
@benschwarz benschwarz closed this Sep 21, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.