Fix handling of decimal numbers in cleanupModernColors.

Author: EreMaijala

src/CSS.php

@@ -625,10 +625,10 @@ protected function cleanupModernColors($content)
         $tag = '(rgb|hsl|hwb|(?:(?:ok)?(?:lch|lab)))';
 
         // remove alpha channel if it's pointless ..
-        $content = preg_replace('/' . $tag . '\(\s*([^\s]+)\s+([^\s]+)\s+([^\s]+)\s+\/\s+1(?:(?:\.\d?)*|00%)?\s*\)/i', '$1($2 $3 $4)', $content);
+        $content = preg_replace('/' . $tag . '\(\s*([^\s)]+)\s+([^\s)]+)\s+([^\s)]+)\s+\/\s+1(?:(?:\.\d?)*|00%)?\s*\)/i', '$1($2 $3 $4)', $content);
 
         // replace `transparent` with shortcut ..
-        $content = preg_replace('/' . $tag . '\(\s*[^\s]+\s+[^\s]+\s+[^\s]+\s+\/\s+0(?:[\.0%]*)?\s*\)/i', '#fff0', $content);
+        $content = preg_replace('/' . $tag . '\(\s*[^\s)]+\s+[^\s)]+\s+[^\s)]+\s+\/\s+0(?:[\.0%]*)?\s*\)/i', '#fff0', $content);
 
         return $content;
     }

tests/CSS/CSSTest.php

@@ -902,6 +902,24 @@ public static function dataProvider()
             '.a1::after{content:"2"}.a1::after{content:"1"}',
         );
 
+        // https://github.com/matthiasmullie/minify/issues/435 - cleanupModernColors
+        $tests[] = array(
+            '.test{color:rgb(1,2,3);background-color:rgba(4,5,6,7)}',
+            '.test{color:#010203;background-color:rgba(4,5,6,7)}',
+        );
+        $tests[] = array(
+            '.test{color:rgb(1, 2, 3); background-color:rgba(4, 5, 6, 7)}',
+            '.test{color:#010203;background-color:rgba(4,5,6,7)}',
+        );
+        $tests[] = array(
+            '.test2{color:rgb(1.5,2.5,3.5);background-color:rgba(0,0,0,0)}',
+            '.test2{color:rgb(1.5,2.5,3.5);background-color:#fff0}',
+        );
+        $tests[] = array(
+            '.test2 { color: rgb(1.5  ,  2.5, 3.5);  background-color:  rgba(0,0, 0,0)}',
+            '.test2{color:rgb(1.5,2.5,3.5);background-color:#fff0}',
+        );
+
         return $tests;
     }