andreiglingeanu opened a new issue #1971: rewrite_css filter minifies CSS 
Custom Properties
URL: https://github.com/apache/incubator-pagespeed-mod/issues/1971
 
 
   Hello,
   
   With the `rewrite_css` filter enabled, [CSS Custom 
Property](https://developer.mozilla.org/en-US/docs/Web/CSS/--*) definitions are 
broken -- their definition is lowercased. This is wrong because the css 
variables [are case sensitive](https://drafts.csswg.org/css-variables/#apis). 
That is variables `--test` and `--Test` are two different variables.
   
   Inline `<style>` tag before:
   
   ```
   <style>
        :root {
                --testMeNow: 123px;
        }
   </style>
   ```
   
   Inline `<style>` after:
   
   ```
   <style>:root{--testmenow:123px}</style>
   ```
   
   This obviously brokes the page because in actual CSS files the variables are 
referenced with `var(--testMeNow)` (no lowercase).
   
   Now I understand that this issue might be caused by the 
[`css_parser`](https://github.com/apache/incubator-pagespeed-mod/tree/master/third_party/css_parser/src)
 and please let me know if I have to redirect the report somewhere else -- I 
don't mind doing the legwork.
   
   The mod-pagespeed version is `1.13.35.2-0` (checked from the 
`X-Mod-Pagespeed` response header).
   
   Please advice on how to better move forward with this. Obviously, I would 
love to keep the `rewrite_css` filter enabled but because of this issue I have 
to keep it off for now.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to