removeComments affects htmlmin:ignore comments

diff --git a/src/htmlminifier.js b/src/htmlminifier.js
index 6d07307..4833dcf 100644 (file)
--- a/src/htmlminifier.js
+++ b/src/htmlminifier.js
@@ -623,7 +623,9 @@
 
         if (/^\s*htmlmin:ignore/.test(text)) {
           isIgnoring = !isIgnoring;
-          buffer.push('<!--' + text + '-->');
+          if (!options.removeComments) {
+            buffer.push('<!--' + text + '-->');
+          }
           return;
         }
         if (options.removeComments) {

diff --git a/tests/minifier.js b/tests/minifier.js
index 7fba30c..77668b6 100644 (file)
--- a/tests/minifier.js
+++ b/tests/minifier.js
@@ -993,7 +993,16 @@
     output = '<!-- htmlmin:ignore --><div class="blah" style="color: red">\n   test   <span> <input disabled/>  foo </span>\n\n   </div><!-- htmlmin:ignore -->' +
             '<div class="blah" style="color: red">test <span><input disabled="disabled"> foo</span></div>';
 
-    equal(minify(input, { collapseWhitespace: true }), output);
+    equal(minify(input, { collapseWhitespace: true, removeComments: false }), output);
+
+    output = '<div class="blah" style="color: red">\n   test   <span> <input disabled/>  foo </span>\n\n   </div>' +
+            '<div class="blah" style="color: red">test <span><input disabled="disabled"> foo</span></div>';
+
+    equal(minify(input, { collapseWhitespace: true, removeComments: true }), output);
+
+    input = '<!-- htmlmin:ignore --><!-- htmlmin:ignore -->';
+    equal(minify(input, { removeComments: true }), '');
+    equal(minify(input, { removeComments: false }), input);
   });
 
   test('meta viewport', function() {