google-code-export / wro4j

Automatically exported from code.google.com/p/wro4j
1 stars 1 forks source link

cssImport and cssDataUri do not go well together #772

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Start build process for a project with configuration as follows:
 - single css file with import directive. An imported css should have the style which have background image.
 - wro.properties:
preProcessors=cssImport,cssDataUri
postProcessors=googleClosureSimple
 - wro.xml:
<groups xmlns="http://www.isdc.ro/wro">
    <group name='client-min'>
        <css>/static/css/client.css</css>
    </group>
</groups>

What is the expected output? What do you see instead?
ER: Merged output css file should have correct style ending.
AR: Merged output css file have broken css styles for styles which meet 
conditions:
  - style was imported to main file
  - style was processed by cssDataUri processor
  - encoded base64 stream has two sequential slashes (//)

Take a look on generated \testa\target\testa\client-min.css. The style 
icon_buggy has broken base64 stream. It ends with 'v/9+'. Following attribute 
suddenly appeared: 'height: 16px' with no semicolons. If you encode original 
image testa\target\testa\static\img\common\image.gif you would see that it 
continues with 'v/9+//'

I have attached packed eclipse project. It reproduces the bug.

What version of the product are you using? On what operating system?
OS: Windows 7 x64 Pro
jdk: 1.7.0_10 (x64)
wro4j: maven build time plugin ver 1.6.3
Eclipse: Kepler 4.2
Maven: Eclipse embedded 3.0.4/1.4.0.20130531-2315

Original issue reported on code.google.com by dms.pop3...@gmail.com on 12 Aug 2013 at 4:38

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by alex.obj...@gmail.com on 13 Aug 2013 at 7:36

GoogleCodeExporter commented 9 years ago
The problem is reproducible with 1.6.3, but seems to be fixed in 1.7.0. 

Resolution: fixed in 1.7.0 (please upgrade).

Original comment by alex.obj...@gmail.com on 15 Aug 2013 at 9:27