Fix empty elements

prettier · Aug 25, 2020 · b44f7e0 · b44f7e0
1 parent 7e6d776
commit b44f7e0
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 28 deletions.
diff --git a/src/language-html/printer-html.js b/src/language-html/printer-html.js
@@ -87,9 +87,9 @@ function embed(path, print, textToDoc, options) {
         return concat([
           printOpeningTagPrefix(node, options),
           group(printOpeningTag(path, options, print)),
-          hardline,
+          doc === "" ? "" : hardline,
           doc,
-          hardline,
+          doc === "" ? "" : hardline,
           printClosingTag(node, options),
           printClosingTagSuffix(node, options),
         ]);

diff --git a/tests/vue/multiparser/__snapshots__/jsfmt.spec.js.snap b/tests/vue/multiparser/__snapshots__/jsfmt.spec.js.snap
@@ -110,9 +110,7 @@ printWidth: 80
 =====================================input======================================
 <custom lang="markdown"></custom>
 =====================================output=====================================
-<custom lang="markdown">
-
-</custom>
+<custom lang="markdown"></custom>
 
 ================================================================================
 `;
@@ -127,14 +125,12 @@ printWidth: 80
  
 </custom>
 =====================================output=====================================
-<custom lang="markdown">
-
-</custom>
+<custom lang="markdown"></custom>
 
 ================================================================================
 `;
 
-exports[`snippet: new line format 2`] = `
+exports[`snippet: non-space format 1`] = `
 ====================================options=====================================
 parsers: ["vue"]
 printWidth: 80
@@ -159,9 +155,7 @@ printWidth: 80
 =====================================input======================================
 <custom lang="markdown">   </custom>
 =====================================output=====================================
-<custom lang="markdown">
-
-</custom>
+<custom lang="markdown"></custom>
 
 ================================================================================
 `;

diff --git a/tests/vue/multiparser/jsfmt.spec.js b/tests/vue/multiparser/jsfmt.spec.js
@@ -1,18 +1,24 @@
-run_spec({dirname: __dirname, snippets: [
+run_spec(
   {
-    name: "empty",
-    code: '<custom lang="markdown"></custom>'
+    dirname: __dirname,
+    snippets: [
+      {
+        name: "empty",
+        code: '<custom lang="markdown"></custom>',
+      },
+      {
+        name: "spaces",
+        code: '<custom lang="markdown">   </custom>',
+      },
+      {
+        name: "new line",
+        code: '<custom lang="markdown">\n \n</custom>',
+      },
+      {
+        name: "non-space",
+        code: '<custom lang="markdown">\n \u2005 \n</custom>',
+      },
+    ],
   },
-  {
-    name: "spaces",
-    code: '<custom lang="markdown">   </custom>'
-  },
-  {
-    name: "new line",
-    code: '<custom lang="markdown">\n \n</custom>'
-  },
-  {
-    name: "new line",
-    code: '<custom lang="markdown">\n \u2005 \n</custom>'
-  },
-]}, ["vue"]);
+  ["vue"]
+);