Merge pull request #970 from mathiasbynens/doctype

Use <!doctypehtml> (without the space) where possible
kangax · Oct 23, 2018 · f625991 · f625991
2 parents cf55c51 + 7e8406e
commit f625991
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/htmlminifier.js b/src/htmlminifier.js
@@ -1238,7 +1238,9 @@ function minify(value, options, partialMarkup) {
       buffer.push(text);
     },
     doctype: function(doctype) {
-      buffer.push(options.useShortDoctype ? '<!doctype html>' : collapseWhitespaceAll(doctype));
+      buffer.push(options.useShortDoctype ? '<!doctype' +
+        (options.removeTagWhitespace ? '' : ' ') + 'html>' :
+        collapseWhitespaceAll(doctype));
     },
     customAttrAssign: options.customAttrAssign,
     customAttrSurround: options.customAttrSurround

diff --git a/tests/minifier.js b/tests/minifier.js
@@ -410,6 +410,11 @@ QUnit.test('doctype normalization', function(assert) {
   assert.equal(minify(input, { useShortDoctype: false }), input);
   assert.equal(minify(input, { useShortDoctype: true }), output);
 
+  assert.equal(minify(input, {
+    useShortDoctype: true,
+    removeTagWhitespace: true
+  }), '<!doctypehtml>');
+
   input = '<!DOCTYPE\nhtml>';
   assert.equal(minify(input, { useShortDoctype: false }), '<!DOCTYPE html>');
   assert.equal(minify(input, { useShortDoctype: true }), output);