varshluck / closure-stylesheets

Automatically exported from code.google.com/p/closure-stylesheets
Apache License 2.0
0 stars 0 forks source link

minification looses trailing space in content:"> " #32

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. create a one-line css file "bug.css", containing only the following line:
    LI.trail:before { content: "> "; }
2. run losure-stylesheets as follows:
    java -jar closure-stylesheets.jar bug.css
3. get the following output:
    LI.trail:before{content:"\00003e "}

There are several issues with the output:
- The space after "\00003e" needs to be doubled.  See section 4.1.4
  of http://www.w3.org/TR/CSS21/syndata.html for reference:

    Only one white space character is ignored after a hexadecimal
    escape. Note that this means that a "real" space after the escape
    sequence must be doubled.

  This problem causes html pages using the minified style sheet
  to render differently from pages which use the original style sheet.
- The trailing zeros in "\00003e" seem unnecessary.  Four characters
  could be saved by writing "\3e" instead.
- Does the ">" need escaping at all?

The problem is present in current git (commit id
86bb800d79a32bfc0328bbf74bfeb772c6eb73b0)

I hope this helps,
Jochen

Original issue reported on code.google.com by jochen.voss@gmail.com on 22 Mar 2013 at 3:17