Skip to content
This repository was archived by the owner on Sep 10, 2019. It is now read-only.

Feature/css condense#460

Open
tatemz wants to merge 4 commits intozurb:masterfrom
tatemz:feature/css-condense
Open

Feature/css condense#460
tatemz wants to merge 4 commits intozurb:masterfrom
tatemz:feature/css-condense

Conversation

@tatemz
Copy link
Contributor

@tatemz tatemz commented Feb 16, 2015

See #440

@gakimball
Copy link
Contributor

What's the purpose of running the uncompressed file through css-condense? Is it just for stripping out comments?

@tatemz
Copy link
Contributor Author

tatemz commented Feb 16, 2015

We can leave comments in as well. Primarily, it condenses media queries (though performance doesn't really matter on a build version).

@gakimball
Copy link
Contributor

Actually, I'd like the CSS-style comments to be stripped out as well if that's possible. We're looking to get into automatically-generated documentation, and for doing HTML-specific documentation we might use Hologram, which requires you to use CSS-style comments to enclose documentation.

I was actually looking for a CSS minifying tool that would just strip out comments and not do any other compression. Sass won't remove them unless it's also compressing the file.

@tatemz
Copy link
Contributor Author

tatemz commented Feb 16, 2015

That'd be a good feature. Along with stripping out comments, css-condense does rearrange the order of sass styles that are imported due to the fact that it assumes media queries should be at the bottom of the stylesheet. It also reorders the selectors in by order of character limits (a takes less processing than div and therefore should be first to end a query sooner).

@gakimball gakimball added the sass label Feb 17, 2015
@gakimball gakimball removed their assignment Mar 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

2 participants