Change doctype to lowercase

wooorm · wooorm · commit 3eb520fc4395 · 2018-06-24T13:21:06.000+02:00
Previously, The `DOCTYPE`, `SYSTEM`, and `PUBLIC` parts of the doctype
were uppercased, however, since the rest of HTML is typically written
lowercase, in my opinion it doesn’t make sense to use uppercase for
only this exception where lowercase works perfectly fine.
diff --git a/lib/doctype.js b/lib/doctype.js
@@ -6,7 +6,7 @@ module.exports = doctype
 function doctype(ctx, node) {
   var pub = node.public
   var sys = node.system
-  var val = '<!DOCTYPE'
+  var val = '<!doctype'
 
   if (!node.name) {
     return val + '>'
@@ -15,9 +15,9 @@ function doctype(ctx, node) {
   val += ' ' + node.name
 
   if (pub != null) {
-    val += ' PUBLIC ' + smart(pub)
+    val += ' public ' + smart(pub)
   } else if (sys != null) {
-    val += ' SYSTEM'
+    val += ' system'
   }
 
   if (sys != null) {
diff --git a/test/doctype.js b/test/doctype.js
@@ -7,13 +7,13 @@ var to = require('..')
 test('`text`', function(t) {
   t.deepEqual(
     to(u('doctype')),
-    '<!DOCTYPE>',
+    '<!doctype>',
     'should stringify doctypes without `name`'
   )
 
   t.deepEqual(
     to(u('doctype', {name: 'html'})),
-    '<!DOCTYPE html>',
+    '<!doctype html>',
     'should stringify doctypes with `name`'
   )
 
@@ -24,7 +24,7 @@ test('`text`', function(t) {
         public: '-//W3C//DTD XHTML 1.0 Transitional//EN'
       })
     ),
-    '<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN">',
+    '<!doctype html public "-//W3C//DTD XHTML 1.0 Transitional//EN">',
     'should stringify doctypes with a public identifier'
   )
 
@@ -35,7 +35,7 @@ test('`text`', function(t) {
         system: 'about:legacy-compat'
       })
     ),
-    '<!DOCTYPE html SYSTEM "about:legacy-compat">',
+    '<!doctype html system "about:legacy-compat">',
     'should stringify doctypes with a system identifier'
   )
 
@@ -47,7 +47,7 @@ test('`text`', function(t) {
         system: 'http://www.w3.org/TR/html4/strict.dtd'
       })
     ),
-    '<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01//" "http://www.w3.org/TR/html4/strict.dtd">',
+    '<!doctype html public "-//W3C//DTD HTML 4.01//" "http://www.w3.org/TR/html4/strict.dtd">',
     'should stringify doctypes with both identifiers'
   )
 
@@ -58,7 +58,7 @@ test('`text`', function(t) {
         system: 'taco"'
       })
     ),
-    "<!DOCTYPE html SYSTEM 'taco\"'>",
+    "<!doctype html system 'taco\"'>",
     'should quote smartly'
   )
 
