Tim Peterson Benchmark comparing compression size of Clean-CSS and YUI compressor #71
Conversation
…I compressor, forgot to add my own site stylesheet
…I compressor, forgot to add my own site stylesheet (TP-site.css)
|
Your clean-CSS file doesn't seem minimized, is it? |
|
You're right, it didn't minimize it. Though b/c it did shrink it somewhat it seemed that Clean-CSS did something. Not sure what was going on so that's when/why I started this issue... |
|
@tim-peterson Hmm, that's very interesting. I'll merge your PR and then investigate what's going on. It seems like a bug, however GoalSmashers.com also has such big CSS files and minimization works fine there. Thanks for your effort! |
Tim Peterson Benchmark comparing compression size of Clean-CSS and YUI compressor
|
@tim-peterson - I ran the minimization of TP-demo locally and it minimizes correctly (size is ~246kB). Could you please share what environment (Unix, Windows), node.js version, and clean-css version ( |
|
@GoalSmashers: Don't get me wrong, but this PR serves no purpose and it's useless for everyone else. It's just garbage files added in the repository, |
|
Please treat it as work in progress - we have a file from Tim which will serve as a basis for benchmarks. Don't worry - at the end we will clean up the repository. On 21 Feb 2013, at 10:39, XhmikosR notifications@github.com wrote:
|
|
Perhaps my environment is the problem. Should the node and cleancss versions match up? MacOSX 10.7.5 I admit I really know nothing about using Node. I'm pretty sure I installed Node awhile back to play with something else. I didn't install it to run Cleancss. All I did in using Cleancss was to download it onto my computer and use it as is. That now seems likely not sufficient and I apologize for wasting your time if that's the case. The only thing I might have in my defense is that other beginner Node programmers would seem to likely to do the same thing (no errors = success, until its not) and so improving the README on the installation part might be a good idea. |
|
They have nothing to do with each other - node.js is at 0.8.x and cleancss at 0.10.1 now and lots have changed since our 0.4.x branch. Could you please update node first and then cleancss with 'npm update -g clean-css' and give it another try? Let me know if you need any assistance! On 21 Feb 2013, at 15:10, tim peterson notifications@github.com wrote:
|
Tim Peterson Benchmark comparing compression size of Clean-CSS and YUI compressor