Added isIgnoredComment()

Patch for issue #70.

diff --git a/dist/all.js b/dist/all.js
index 4268f53..c702659 100644 (file)
--- a/dist/all.js
+++ b/dist/all.js
@@ -386,7 +386,7 @@
 
   function collapseWhitespaceSmart(str, prevTag, nextTag) {
     // array of tags that will maintain a single space outside of them
-    var tags = ['a', 'b', 'big', 'button', 'code', 'em', 'font', 'i', 'kbd', 'mark', 'q', 's', 'small', 'span', 'strike', 'strong', 'sub', 'sup', 'tt', 'u'];
+    var tags = ['a', 'abbr', 'acronym', 'b', 'big', 'button', 'code', 'del', 'dfn', 'em', 'font', 'i', 'ins', 'kbd', 'mark', 'q', 's', 'small', 'span', 'strike', 'strong', 'sub', 'sup', 'tt', 'u', 'var'];
 
     if (prevTag && prevTag !== 'img' && (prevTag.substr(0,1) !== '/'
       || ( prevTag.substr(0,1) === '/' && tags.indexOf(prevTag.substr(1)) === -1))) {
@@ -410,6 +410,10 @@
     return ((/\[if[^\]]+\]/).test(text) || (/\s*(<!\[endif\])$/).test(text));
   }
 
+  function isIgnoredComment(text) {
+    return (/^!/).test(text);
+  }
+
   function isEventAttribute(attrName) {
     return (/^on[a-z]+/).test(attrName);
   }
@@ -742,6 +746,8 @@
         if (options.removeComments) {
           if (isConditionalComment(text)) {
             text = '<!--' + cleanConditionalComment(text) + '-->';
+          } else if (isIgnoredComment(text)) {
+            text = '<!--' + text + '-->';
           }
           else {
             text = '';

diff --git a/src/htmlminifier.js b/src/htmlminifier.js
index b4bced9..bf38f30 100644 (file)
--- a/src/htmlminifier.js
+++ b/src/htmlminifier.js
@@ -66,6 +66,10 @@
     return ((/\[if[^\]]+\]/).test(text) || (/\s*(<!\[endif\])$/).test(text));
   }
 
+  function isIgnoredComment(text) {
+    return (/^!/).test(text);
+  }
+
   function isEventAttribute(attrName) {
     return (/^on[a-z]+/).test(attrName);
   }
@@ -398,6 +402,8 @@
         if (options.removeComments) {
           if (isConditionalComment(text)) {
             text = '<!--' + cleanConditionalComment(text) + '-->';
+          } else if (isIgnoredComment(text)) {
+            text = '<!--' + text + '-->';
           }
           else {
             text = '';