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

Feature/css condense #460

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Feature/css condense #460

wants to merge 4 commits into from

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