Skip to content

Bug fix: - #13

Closed
Zh-L wants to merge 1 commit into
tubalmartin:masterfrom
Zh-L:feature
Closed

Zh-L wants to merge 1 commit into
tubalmartin:masterfrom
Zh-L:feature

Conversation

@Zh-L

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

Copy link
Copy Markdown
  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
Copy Markdown
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
Copy Markdown
Owner

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

@tubalmartin

Copy link
Copy Markdown
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