Fix tight empty SVG close after unquoted attribute

Author: wooorm

lib/element.js

@@ -21,6 +21,7 @@ var greaterThan = '>'
 var slash = '/'
 
 // Stringify an element `node`.
+// eslint-disable-next-line complexity
 function element(ctx, node, index, parent) {
   var parentSchema = ctx.schema
   var name = node.tagName
@@ -31,6 +32,7 @@ function element(ctx, node, index, parent) {
   var root = node
   var content
   var attrs
+  var last
 
   if (parentSchema.space === 'html' && name === 'svg') {
     ctx.schema = svg
@@ -64,7 +66,15 @@ function element(ctx, node, index, parent) {
     value = lessThan + name + (attrs ? space + attrs : '')
 
     if (selfClosing && close) {
-      if (!ctx.tightClose || attrs.charAt(attrs.length - 1) === slash) {
+      last = attrs.charAt(attrs.length - 1)
+      if (
+        !ctx.tightClose ||
+        last === slash ||
+        (ctx.schema.space === 'svg' &&
+          last &&
+          last !== quotationMark &&
+          last !== apostrophe)
+      ) {
         value += space
       }
 

test/element.js

@@ -37,6 +37,17 @@ test('`element`', function(t) {
     'should stringify voids with `/` in `closeSelfClosing` and `tightSelfClosing` mode'
   )
 
+  // This works in a browser.  The `/` is not part of the `[src]`.
+  t.deepEqual(
+    to(h('img', {src: 'index.jpg'}), {
+      preferUnquoted: true,
+      closeSelfClosing: true,
+      tightSelfClosing: true
+    }),
+    '<img src=index.jpg/>',
+    'should stringify voids with `/` in `closeSelfClosing` and `tightSelfClosing` mode, without space after an unquoted attribute'
+  )
+
   t.deepEqual(
     to(h('img', {title: '/'}), {
       preferUnquoted: true,

test/svg.js

@@ -38,7 +38,18 @@ test('svg', function(t) {
       tightSelfClosing: true
     }),
     '<circle/>',
-    'should stringify voids with `/` in `closeEmptyElements` and `tightSelfClosing` mode'
+    'should stringify empties with `/` in `closeEmptyElements` and `tightSelfClosing` mode'
+  )
+
+  // `<circle cx=2 cy=2 r=1/>` does not work in browsers.  Needs a space.
+  t.deepEqual(
+    to(s('svg', {viewBox: '0 0 4 4'}, s('circle', {cx: 2, cy: 2, r: 1})), {
+      preferUnquoted: true,
+      closeEmptyElements: true,
+      tightSelfClosing: true
+    }),
+    '<svg viewBox="0 0 4 4"><circle cx=2 cy=2 r=1 /></svg>',
+    'should stringify empties with `/` in `closeEmptyElements` and `tightSelfClosing` mode, *with* a space after an unquoted attribute'
   )
 
   t.deepEqual(