scss fixes #347

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@gajeev

nesting elements in sass implies that they are children

@parndt
Refinery member

Are .flash_message and .flash_notice not children of .flash?

@parndt
Refinery member

Hm, yeah, I think those are supposed to be like:

.flash {
  &.flash_notice {
     //etc
   }
  &.flash_message {
    // etc
  }
}
@gajeev

no they're not children, yeah i think your next comment will work, do you want me to re-do it like that?

@parndt
Refinery member

Yes please if you wouldn't mind :)

@gajeev gajeev closed this Sep 5, 2013
@parndt
Refinery member

Why'd you close it? You just have to push new commits to the same branch :)

@parndt parndt reopened this Sep 5, 2013
@gajeev

oops :)

@parndt parndt referenced this pull request Sep 5, 2013
Merged

scss fixes #348

@parndt parndt closed this Sep 5, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment