Closed Phunky closed 11 years ago
+1 I like it, would love to see it in master branch!
Instead of compiling both silent and visible classes, you could use a boolean to enable either one or the other. I guess it would be better for the final output.
Very true @HugoGiraudel and that's actually what I did myself after this PR, although for initial prototyping its good to have both so you can chop & change live in the browser while testing.
First of all, don't include changes that aren't related to the PR. By that I mean the change of filename and comments. Read this.
Second of all, what's the benefit of doing this? I don't see any.
The benefit is, that you don't have to talk about the layout in the markup. Instead of saying "This column is this wide", you can say "this is the navigation column" and specify the style for it where it belongs - in the style sheets.
As for the PR, I agree. That could've been two PRs. I appreciate that suggested changes though. Single line comments will not appear in the compiled CSS, underscored SCSS files will not clutter your target directory.
Maybe we can split this PR into two separate ones?
Going to close this because: it isn't quite aligned with the point of this module, but it also includes unrelated changes and doesn't merge cleanly.
Added @extend selectors to griddle to remove the need for placing griddle class within the HTML.
Extremely useful for responsive designs that don't want to place specific presentation classes within the markup.