New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rewrite_css removes transform values #66

Closed
GoogleCodeExporter opened this Issue Apr 6, 2015 · 2 comments

Comments

Projects
None yet
1 participant
@GoogleCodeExporter

GoogleCodeExporter commented Apr 6, 2015

Using 0.9.0.0-128, the rewrite_css filter messes up transform values for things 
like scale, skew, rotate, and translate.

This CSS:

-moz-transform: rotate(7deg);
-webkit-transform: rotate(7deg);
-moz-transform: skew(-25deg);
-webkit-transform: skew(-25deg);
-moz-transform: scale(0.5);
-webkit-transform: scale(0.5);
-moz-transform: translate(3em, 0);
-webkit-transform: translate(3em, 0);


is changed to be this:

-moz-transform:rotate();-webkit-transform:rotate();-moz-transform:skew();-webkit
-transform:skew();-moz-transform:scale();-webkit-transform:scale();-moz-transfor
m:translate();-webkit-transform:translate()


Thanks!

Original issue reported on code.google.com by joe.lencioni on 10 Nov 2010 at 5:28

@GoogleCodeExporter

This comment has been minimized.

GoogleCodeExporter commented Apr 6, 2015

You are using the first release of mod_pagespeed.  There's been an important 
bug-fix release.  Please download the new version from 
http://code.google.com/speed/page-speed/download.html

That binary distribution avoids this problem by temporarily removing the 
"rewrite_css" filter from the Core set.

The particular bug you identified has already been identified and fixed in the 
rewrite_css filter, but has not yet made it to a binary release.

Original comment by jmara...@google.com on 10 Nov 2010 at 6:17

@GoogleCodeExporter

This comment has been minimized.

GoogleCodeExporter commented Apr 6, 2015

Verified this is fixed in svn trunk. I've added a test to make sure.

Original comment by sligocki@google.com on 10 Nov 2010 at 6:58

  • Changed state: Fixed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment