search

inuitcss / getting-started

[DEPRECATED] [go to intuitcss/inuitcss] Getting stuck into inuitcss has never been simpler
https://github.com/inuitcss/inuitcss
351 stars 12 forks source link

Both Class and Placeholder defined #7

Closed llexical closed 9 years ago

llexical commented 9 years ago

Not sure if this is the correct place to post this, so please say if there is something else. But I was wondering why you are defining both placeholders and classes. It seems to me a placeholder is exactly the same as a class in behaviour except for the fact it will not be added into the final css. Adding a class completely negates this as it will now render as a class. Would love to hear your thoughts!

csswizardry commented 9 years ago

It’s basically because of this: http://csswizardry.com/2014/01/extending-silent-classes-in-sass/

If people are going to use @extend (I’d rather they didn’t) then it’s best they only @extend the silent version. I might take the silent classes back out soon—doubles up on maintenance/typing/name-changes etc. :(

nenadjelovac commented 9 years ago

@Lilibuth12 you can read more about removing placeholder classes on this topic. Please feel free to join!