YUI Compressor :: {3} Changes CheckedIn - available to pull from Github

Stray semicolon inserted into CSS file

In CSS files that use media queries and a comment to hide rules from IE5 Mac. Where the comment is used in the CSS file before the closing bracket of the @media statement a stray semicolon is inserted in-between the closing comment and the closing bracket.

Before:
@media screen {
/* for IE 5.x-6, hidden from IE 5 Mac */ /**/
* html div#page {
height: 1%;
}
/**/ /* end hidden from IE 5 Mac */
}

After:
@media screen{/**/ * html div#page{height:1%;}/**/;}

Expected
@media screen{/**/ * html div#page{height:1%;}/**/}

Compressor breaks on this link-icons.css file

Specifically I think it has trouble with the multiple selectors that use ^...

/* --------------------------------------------------------------

link-icons.css
* Icons for links based on protocol or file type.

See the Readme file in this folder for additional instructions.

-------------------------------------------------------------- */

/* Use this class if a link gets an icon when it shouldn't. */
body a.noicon { background:none; padding:0; margin:0; }

/* Make sure the icons are not cut */
a[href^="http:"], a[href^="mailto:"], a[href^="http:"]:visited,
a[href$=".pdf"], a[href$=".doc"], a[href$=".xls"], a[href$=".rss"],
a[href$=".rdf"], a[href^="aim:"] {
padding:2px 22px 2px 0;
margin:-2px 0;
background-repeat: no-repeat;
background-position: right center;
}

/* External links */
a[href^="http:"] { background-image: url(icons/external.png); }
a[href^="mailto:"] { background-image: url(icons/email.png); }
a[href^="http:"]:visited { background-image: url(icons/visited.png); }

/* Files */
a[href$=".pdf"] { background-image: url(icons/pdf.png); }
a[href$=".doc"] { background-image: url(icons/doc.png); }
a[href$=".xls"] { background-image: url(icons/xls.png); }

/* Misc */
a[href$=".rss"],
a[href$=".rdf"] { background-image: url(icons/feed.png); }
a[href^="aim:"] { background-image: url(icons/im.png); }

Replacement of CSS hack html>/**/body

Because YUI Compressor (2.3.5.) replaces:

html>/**/body (valid CSS hack for modern browsers expect IE 7)

with:

html>body

it removes the hack.

errors should be sent to the error stream

When there is an error (for example no arguments or bad arguments) yuicompressor output the help and usage to the output stream (/dev/stdout on unix/mac).
All error/help/warning output should be sent to the error stream instead (/dev/stderr on unix/mac).

Reproduce :
java -jar yuicompressor-2.3.5.jar --badparameter 2> /dev/null

Result :
Output the usage help on the terminal

Wanted result:
Usage help and potential errors are redirected to /dev/null

Missed CSS optimization - unneeded trailing ;

The last ; in a CSS class is optional.
Even if I remove it in my CSS source file, the compressor set it back.

The --preserve-semi option available for JavaScript compression should also exist for the CSS compression.

By default I recommend to remove all the trailing ;

Example:

div {color: black;}

Should be

div{color:black}

In the web application I'm working on it's about 500 ; that can be removed.

"$" signs (common in VSS)breaks compressor for CSS

If there are headers like this in the CSS source:

The compressor will break and throw a:

Exception in thread "main" java.lang.reflect.InvocationTargetException
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at com.yahoo.platform.yui.compressor.Bootstrap.main(Bootstrap.java:20)
Caused by: java.lang.IllegalArgumentException: Illegal group reference
at java.util.regex.Matcher.appendReplacement(Unknown Source)
at com.yahoo.platform.yui.compressor.CssCompressor.compress(CssCompressor.java:86)
at com.yahoo.platform.yui.compressor.YUICompressor.main(YUICompressor.java:178)
... 5 more

import statement stripped out of css file

If the first CSS rule is empty, but there is an import statement before it, the import statement is stripped out of the compressed version. I tried this with version 2.4.1 and it doesn't work. If you comment out the empty rule, everything is fine.

compressor removes beginning of css

Imagine simple css:

@charset 'utf-8';
@media all {
body {
}
body {
background-color: gold;
}
}

compress it:
java -jar yuicompressor-2.4.2.jar sample.css -o sample-min.css

result:
body{background-color:gold;}}

expected:
@charset 'utf-8';@media all {body{background-color:gold;}}

Byte Order Marker and Default Charset in Vista (x64) and XP (x86)

Using YUIcompressor on Vista SP1(x64) and XP SP3(x86) with JavaScript files created by Visual Studio 2008 results in errors due to the 3-byte Byte Ordering Mark at the beginning of the file. YUIcompressor defaults to charset Cp1252, and the EF BB EB Byte Ordering Mark at the beginning of the JavaScript files result in a very unobvious YUIcompressor failure. Simply deleting the 3 bytes makes the files work as does adding
--charset UTF-8
to the YUIcompressor command line.
I don't know if this is a feature request or a bug, but on opening a file, could YUIcompressor inspect the first characters of the file for a known BOM and if it finds one, use the appropriate charset unless overridden by the --charset option?

Empty class declaration can break @media section

An empty class declaration can break an @media section

SOURCE:
@media print {
.noprint { display: none; }
}

@media screen {
.breakme {}
.printonly { display: none; }
}

RESULT:
@media print{.noprint{display:none;}
}

.printonly{display:none;}
}

EXPECTED:
@media print{.noprint{display:none;}
}

@media screen{.printonly{display:none;}
}

Ticket 2495387 may be a related issue

A workaround is to remove the redundant declaration or to ensure the first declaration in an @media section actually declares something

Java exception while processing .js file

Running YUICompressor over a particular .js file is throwing a Java exception during processing.
You can retrieve the CSS file in question here: http://www.zoeybot.com/construct.css

yuicompressor throws Java exception during use.

The yuicompressor throws Java exception while processing the attached javascript file.
See Test Information

Improve charset handling to try to avoid org.mozilla.javascript.EvaluatorException

A lot of people are having compressor fail with an org.mozilla.javascript.EvaluatorException, and it appears that it is most commonly happening when the charset option is not provided and the file is encoded in utf-8. If compressor is unable to detect the correct charset, an approach that might work is to catch that exception, retry the operation with utf-8 encoding, and if it fails again, rethrow the original error with additional advice about providing the correct charset to the compressor.

CSS file that makes yuicompressor crash

java -jar yuicompressor-2.4.2.jar css/a.css
Exception in thread "main" java.lang.reflect.InvocationTargetException
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:616)
at com.yahoo.platform.yui.compressor.Bootstrap.main(Bootstrap.java:20)
Caused by: java.lang.IllegalArgumentException: Illegal group reference
at java.util.regex.Matcher.appendReplacement(Matcher.java:725)
at com.yahoo.platform.yui.compressor.CssCompressor.compress(CssCompressor.java:86)
at com.yahoo.platform.yui.compressor.YUICompressor.main(YUICompressor.java:178)
... 5 more

Suffix attribute selector + pseudo selector makes YUI Compressor crash

Title says it all. See the test information for details.

Preserving CSS comments loses formatting

When using the */! style comments in css, line breaks are stripped out when the expected behavior would be to preserve them in the same way as is done in the js files. This is especially important for licensing comments, where readability is necessary. The few extra characters of savings make it difficult to stay compliant with legal requirements of some licenses.

Expected behavior: comments starting with /*! should be preserved exactly as they were in the source file

Observed behavior: comments are preserved as expected in JavaScript output files. comments lose line breaks in CSS output files, making them less readable.

Also, this ability to preserve comments for cases like licensing should be documented more clearly. I eventually found the technique in the change log.

[bz 2451000] YUI compressor adding semicolon in @import

If I have an import path that contains {}, yui compressor adds a semicolon at the end.

> @import "../${TEST}/test.css";
compresses to
> @import "../${TEST;}/test.css";

YUI Compressor hoses @media type and declarations

@media screen and (min-width: 800px) {

nav { background-color: #fc0; }

}

Basically, removes the space after and and the @media doesn't work at all.

@media min-width: 800px {

}

works fine though!

special comments are not preserved in otherwise empty css file

Special comments are not preserved in otherwise empty css files. For example:



/*! special */

body {

will be minified to the empty string.

Important comment syntax not preserved in JS Minification

YUI compressor uses a modified version of Rhino to detect important comments which have the form /*! ... */. The contents of important comments are preserved and appear in the minified output from YUI compressor. This is a good thing as they are often used to include copyright and licensing information.

Unfortunately, in the JavaScript minification process YUI converts the /*! ... */ comment into a regular /* ... */ comment. This means if you run YUI compressor on a JavaScript file that is already minified it will remove the important comment. This is a growing use case as more developers utilize automated minifers/performance tools these are often chained together or run mutliple times on the same resource at different steps of the development lifecycle.

This behavior is also different from the way YUI Compressor handles important comments in CSS. Important comments in CSS are left in their /*! ... */ format so that multiple CSS minifying with YUI compressor will not remove the important comments.

I've tracked down exactly the issue. It is caused by a single line of code in the modifed version of the class org.mozilla.javascript.TokenStream. Line 724. s1.substring(1) is used to remove the "!" from the contents of the comment. Replacing this line with "this.string = s1;" will solve this issue. I have attached patch for TokenStream.java to fix this bug.

[bz 3218041] Compressor should use UTF-8 as default encoding

The YUI Compressor currently uses the platform encoding if no charset is specified. This is dangerous, as JavaScript
files are often edited and processed on different platforms, and platform encodings are generally incompatible (on my
Mac I see MacRoman, which is hardly ever seen on the internet!).

The tool should use UTF-8 as the default, the required encoding of both the internet and Yahoo:
http://tools.ietf.org/html/rfc2277
[ see bugzilla ticket for link to additional internal documentation ]

As this change may be a compatibility issue for third-party developers who have created JavaScript files with non-ASCII characters relying on a platform encoding other than UTF-8, it should only be made in a major release where a
compatibility break is acceptable. As a workaround for YUI, I have also filed bugzilla ticket 3199463 against the Builder tool.

Final semi-colon in each rule group not removed

If you try to compress CSS it always leaves the last semi-colon before the curly brace. For example the following CSS:
.header {
margin: 0;
background-color: #555753;
}

...comes back as:
.header{margin:0;background-color:#555753;}

...but it should be:
.header{margin:0;background-color:#555753}

Yuicompressor-2.4.2.jar cut @import tag while compressing css files

Yuicompressor-2.4.2.jar cut @import tag while compressing css files, such as: @import url("import1.css"); or @import url "import2.css";

java -jar yuicompressor-2.4.2.jar /home/akula/1.css -o /home/akula/1_min.css

Before compress:
@import url('2.css');
.mLstSearchQuery{}
.mLstSearchForm{ display:block; float:left;}
.mLstFltMenu { background:url(../img/prof/menu/bg.gif)
...

After compress:
.mLstSearchForm{display:block;float:left;}.mLstFltMenu{background:url(../img/prof/menu/bg.gif)
...

Exception thrown when handling specific CSS selector

This fails:



a[href$="/test/"] span:first-child { }

This succeeds:



a[href*="/test/"] span:first-child { }

As does this:



a[href$="/test/"] span { }

space not preserved when compressing CSS that contains attribute selector

I have a CSS directive that looks like this:

span[style*="color: #fff"][style*="background-color: #000"] {
color: #000 !important;
background: none !important;
font-style: italic !important;
}

When this is compressed by YUI Compressor the space is removed from "color: #fff". Without that space it no longer matches what I want. I think any quoted strings inside an attribute selector should probably be left alone.

Fix the crash when compressing CSS files with $ characters inside the comments that needs to be preserved

When there's a need to preserve copyright notices in the CSS files and there's a source-control annotation with $ in the text, the compressor would fail with this error:

java.lang.IllegalArgumentException: Illegal group reference
at java.util.regex.Matcher.appendReplacement(Matcher.java:713)
at com.yahoo.platform.yui.compressor.CssCompressor.compress(CssCompressor.java:86)
at com.amazon.javascript.tools.CompressorTask$CompressionType.compress(CompressorTask.java:161)
at com.amazon.javascript.tools.CompressorTask$CompressionType.access$100(CompressorTask.java:117)
at com.amazon.javascript.tools.CompressorTask.execute(CompressorTask.java:106)
at org.apache.tools.ant.UnknownElement.execute(UnknownElement.java:288)
at sun.reflect.GeneratedMethodAccessor1.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
at java.lang.reflect.Method.invoke(Method.java:597)
at org.apache.tools.ant.dispatch.DispatchUtils.execute(DispatchUtils.java:106)
at org.apache.tools.ant.Task.perform(Task.java:348)
at org.apache.tools.ant.Target.execute(Target.java:357)
at org.apache.tools.ant.Target.performTasks(Target.java:385)
at org.apache.tools.ant.Project.executeSortedTargets(Project.java:1337)
at org.apache.tools.ant.Project.executeTarget(Project.java:1306)
at org.apache.tools.ant.helper.DefaultExecutor.executeTargets(DefaultExecutor.java:41)
at org.apache.tools.ant.Project.executeTargets(Project.java:1189)
at org.apache.tools.ant.Main.runBuild(Main.java:758)
at org.apache.tools.ant.Main.startAnt(Main.java:217)
at org.apache.tools.ant.launch.Launcher.run(Launcher.java:257)
at org.apache.tools.ant.launch.Launcher.main(Launcher.java:104)

This patch is to fix the crash.

YUI Compressor breaks on some combinations of CSS3 attribute selector and pseudo classes

YUI Compressor will throw an exception on a CSS declaration like:

ul[id$=foo] label:hover

I was unable to narrow down the bug any further.

truncates '-webkit-transform-origin' double zero values to single zero value

YUI compressor appears to be truncating double zero settings for the '-webkit-transform-origin' property down to one zero.

'-webkit-transform-origin: 0 0;' => '-webkit-transform-origin:0;'

Workaround is to set the '-x' and '-y' of the property separately.

Configuration.setOutput(String) forces output filename to lowercase [httpserver]

At line 208, the output filename is forced to lowercase. This causes a FileNotFoundException on case-sensitive file systems, when the output filename includes a mixed-case output path, e.g. "clientcide-2.2.0/assets/stickyWinHTML/stickywin.css"

--nomunge option has the opposite effect [httpserver]

The

--nomunge

option now appears to munge identifiers. Omitting the option leaves identifiers unmodified.

@media syntax compression breaks deceleration

YUI Compressor will compress an @media rule such as:

@media all and (max-device-width: 480px)

to:

 @media all and(max-device-width: 480px)

Which---to my knowledge--is invalid, and certainly breaks in Mobile Safari on iOS. The space between "and" and "(" needs to be retained.

CSS selectors must be case-sensitive

ID and class names in CSS selectors should have their case preserved during compression.

YUI Compressor Multiple Files

I use the YUI compressor as a part of a deployment script. When deploying, YUI compressor runs over 30 files and generally takes a few minutes.

I beleive most of the time spent is on loading and unloading the jvm and the jar package.

It would be awesome if YUI compressor could accept multiple files and outpout to multiple files at once.

Eg.

java -jar yuicompressor-x.y.z.jar myfile1.js,myfile2.js -o myfile-min.js,myfile2.js

It doesnt have to have this syntax, but just some mechanism of doing this.

Thanks,

Scott

Remove ;} in CSS

It appears that YUI compressor for css usually creates this type of code:

.test{color:#444;}

While that is fine, it's even better to remove the semicolon and write this:

.test{color:#444}

The result is equally valid but can reduce the filesize quite a bit in larger CSS files.

Currently I have to perform this step with a text editor, and I would very much like to see this built in to YUI compressor.

Thanks,
Steven

bypassing dataurl for faster parsing

css compression is very slow on big file-sizes.
I played arraound with data url background images so my css file had a size of about 140kb and yuicompressor takes 5 min to comress it.
so my idea is to bypass dataurls, maybe this is implementable in a later release of yuicompressor?

this is the data url-example I am using. this backround image line is 90kb?

selector1, #selctor2 {

background: url("data:image/png;base64,iVBORw_looongdata");
}

i have the current css I am playing with in the attachement.

kind regards
tanila

border:none -> border:0

(via phpied.com)
Mathias (http://mathiasbynens.be/) Says:
May 28th, 2010 at 7:56 am

Some additional optimizations:



.test { border:none; }

Compressed:



.test{border:none}

Could be:



.test{border:0}

The same goes for other CSS properties that have a none value but can also take numeric values.

[Pull Request] - Don't bork Safari by removing spaces in front of parens in @media lines.

Pull Request from isaacs
Commit: http://github.com/isaacs/yuicompressor/commit/0e290b8e05392f19bccc2f7d524725a87739e6d6
Files modified:

src/com/yahoo/platform/yui/compressor/CssCompressor.java

[Pull Request] - Add a test script and a functional test for the safari-borking bug reported by rharmes.

Pull Request from isaacs
Commit: http://github.com/isaacs/yuicompressor/commit/04665d549a79f57a9b22d2f640cd9a1d2847ac9f
Files added:

[Pull Request] - Fix whitespace

This fixes the whitespace issue