Skip to content

Conversation

Zh-L
Copy link

@Zh-L Zh-L commented Feb 3, 2014

  1. Bug fix for the class with name ".mg0px { ... }".
    $css = preg_replace('/(^|[^0-9]):(?:0?.)?0(?:em|ex|ch|rem|vw|vh|vm|vmin|cm|mm|in|px|pt|pc|%|deg|g?rad|m?s|k?hz)/iS', '${1}:0', $css);

  2. Bug fix unclosed quotes in filter. Example: progid:DXImageTransform.Microsoft.gradient(startColorstr='u', endColorstr='url(YUICSSMIN_PRESERVED_TOKEN_0);
    $css = preg_replace_callback('/(?:"(?:[^\\";}]|\.|\)")|'."(?:'(?:[^\\';}]|\.|\)')/S", array($this, 'replace_string'), $css);

1) Bug fix for the class with name ".mg0px { ... }".
$css = preg_replace('/(^|[^0-9]):(?:0?\.)?0(?:em|ex|ch|rem|vw|vh|vm|vmin|cm|mm|in|px|pt|pc|%|deg|g?rad|m?s|k?hz)/iS', '${1}:0', $css);

2) Bug fix unclosed quotes in filter. Example: progid:DXImageTransform.Microsoft.gradient(startColorstr='u', endColorstr='url(___YUICSSMIN_PRESERVED_TOKEN_0___);
$css = preg_replace_callback('/(?:"(?:[^\\\\";}]|\\\\.|\\\\)*")|'."(?:'(?:[^\\\\';}]|\\\\.|\\\\)*')/S", array($this, 'replace_string'), $css);
@tubalmartin
Copy link
Owner

I've just fix the first case you mentioned, but I don't understand the second case, could you elaborate? Thank you

@tubalmartin
Copy link
Owner

As I said in #12 the second case is not the compressors fault.

@tubalmartin
Copy link
Owner

First case fixed in v2.4.8-3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants