Skip to content
This repository has been archived by the owner. It is now read-only.

The minification actually makes the stylesheet bigger. #21

Closed
iflan opened this issue Oct 23, 2015 · 1 comment
Closed

The minification actually makes the stylesheet bigger. #21

iflan opened this issue Oct 23, 2015 · 1 comment
Assignees

Comments

@iflan
Copy link
Member

@iflan iflan commented Oct 23, 2015

Originally reported on Google Code with ID 2

What steps will reproduce the problem?

1. Create a input.css file:
---------------
#first, #second, #third {
    width: 100px;
    height: 100px;
    color: #ff0000;
    background-color: #00ff00;
}
#second {
    width: 105px;
}
---------------

2. Run: java -jar closure-stylesheets.jar --output-file output.css input.css

What is the expected output? What do you see instead?

Expected (and YUI Compressor achieves this), 100 bytes:
---------------
#first,#second,#third{width:100px;height:100px;color:#f00;background-color:#0f0}#second{width:105px}
---------------

With Closure I get a 204 byte file instead (original was 138!):
---------------
#first{width:100px;height:100px;color:#f00;background-color:#0f0}#second{height:100px;color:#f00;background-color:#0f0}#third{width:100px;height:100px;color:#f00;background-color:#0f0}#second{width:105px}
---------------

What version of the product are you using? On what operating system?

I'm using the initial release, SHA1 5d8092ec7287f8b05b41ff71f52ff2dc0576f414

Windows 7 64bit SP1

Please provide any additional information below.

This sucks big time.

Reported by xxStrom on 2011-11-15 02:44:40

@iflan iflan self-assigned this Oct 23, 2015
@iflan
Copy link
Member Author

@iflan iflan commented Oct 23, 2015

When gzipped (you are gzipping right?) the size for both the yahoo and google compressed
files is the same: 99B

Does NOT suck big time. I assume google is targeting the best minification for gzipped
files (as they do with the closure js library)

Reported by alvaro@gravitydev.com on 2012-02-04 03:41:05

Loading

@iflan iflan closed this Oct 26, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant