-
Notifications
You must be signed in to change notification settings - Fork 58
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Why not to use csso instead of cssmin? #28
Comments
Because we already have |
@davglass But |
@davglass Sorry, but the difference in logic between Just one simple sample to compare. Source: a {
color: red
}
a {
color: red
}
a{color:red}a{color:red}
a{color:red} |
Sorry, I didn't mean to be short with you. I did compare the two, but I needed a tool that I could trust to not break things and that I could use right away. Our version of If you can build your test suite out and show that your compressor is a better choice in the future, then I will judge them again and make that call then. Until then, I have to go with the one that I can trust to do it's job and do it well. |
I see. Ok, we will try to do it. Thank you. |
You should also take several CSS files from large OSS projects & |
We did it many times. :) CSSO is the "good old whitespace killer" + structure compression. So in most (ok, I will not use the word "all") cases it is better, especially for large (something about 100K) CSS files, especially generated by some CSS code generators. Here is another example to show the difference between compression methods. Source (updated: sorry, it was old one, here is the new one): a {
color: blue;
}
b {
color: #00f
}
a{color:blue}b{color:#00f}
a,b{color:#00f} Yep, you will not see the huge profit on small styles. There are nothing to compress, just spaces, numbers, colors, etc — old tools are good enough to do it. But try to use it on huge files. BTW, if you have another example (CSSO is bad, some tool is good), I will be very glad to see it and to fix it. |
csso
can make structural optimizations thancssmin
can't.Have a look http://bem.info/tools/csso/
The text was updated successfully, but these errors were encountered: