Strip trailing ";" from style attributes (this allows to strip quotes from values...

diff --git a/src/htmlminifier.js b/src/htmlminifier.js
index fd7d940..e42e2db 100644 (file)
--- a/src/htmlminifier.js
+++ b/src/htmlminifier.js
@@ -123,11 +123,12 @@
     else if (attrName === 'class') {
       return collapseWhitespace(trimWhitespace(attrValue));
     }
-    else if (isUriTypeAttribute(attrName, tag)
-          || isNumberTypeAttribute(attrName, tag) 
-          || attrName === 'style') {
+    else if (isUriTypeAttribute(attrName, tag) || isNumberTypeAttribute(attrName, tag)) {
       return trimWhitespace(attrValue);
     }
+    else if (attrName === 'style') {
+      return trimWhitespace(attrValue).replace(/\s*;\s*$/, '');
+    }
     return attrValue;
   }
   

diff --git a/tests/index.html b/tests/index.html
index fab005b..bec13dc 100644 (file)
--- a/tests/index.html
+++ b/tests/index.html
@@ -220,7 +220,11 @@
           equals(minify(input, { cleanAttributes: true }), output);
           
           input = '<p style="    color: red; background-color: rgb(100, 75, 200);  "></p>';
-          output = '<p style="color: red; background-color: rgb(100, 75, 200);"></p>';
+          output = '<p style="color: red; background-color: rgb(100, 75, 200)"></p>';
+          equals(minify(input, { cleanAttributes: true }), output);
+          
+          input = '<p style="font-weight: bold  ; "></p>';
+          output = '<p style="font-weight: bold"></p>';
           equals(minify(input, { cleanAttributes: true }), output);
         });