fix: hug when needed for components and fragments (#397)

* fix: hug when needed for components and fragments * fix: consider Component, Fragments and custom elements as whitespace sensitive * chore: changeset
withastro · Dec 27, 2023 · e97406a · e97406a
1 parent 3d596c1
commit e97406a
Show file tree

Hide file tree

Showing 7 changed files with 21 additions and 21 deletions.
diff --git a/.changeset/khaki-parrots-yawn.md b/.changeset/khaki-parrots-yawn.md
@@ -0,0 +1,5 @@
+---
+'prettier-plugin-astro': minor
+---
+
+Fix plugin sometimes including significant whitespace inside components, fragments and custom elements
diff --git a/src/printer/index.ts b/src/printer/index.ts
@@ -239,7 +239,7 @@ export function print(path: AstPath, opts: ParserOptions, print: printFn): Doc {
 					}
 					if (!hugEnd && lastChild && isTextNode(lastChild)) {
 						if (isInlineElement(path, opts, node) && !didSetEndSeparator) {
-							noHugSeparatorEnd = softline;
+							noHugSeparatorEnd = line;
 						}
 						trimTextNodeRight(lastChild);
 					}

diff --git a/src/printer/utils.ts b/src/printer/utils.ts
@@ -27,18 +27,15 @@ export const dotReplace = 'ωP_';
 export const interrogationReplace = 'ΔP_';
 
 export function isInlineElement(path: AstPath, opts: ParserOptions, node: anyNode): boolean {
-	return node && node.type === 'element' && !isBlockElement(node, opts) && !isPreTagContent(path);
+	return node && isTagLikeNode(node) && !isBlockElement(node, opts) && !isPreTagContent(path);
 }
 
 export function isBlockElement(node: anyNode, opts: ParserOptions): boolean {
 	return (
-		(node &&
-			node.type === 'element' &&
-			opts.htmlWhitespaceSensitivity !== 'strict' &&
-			(opts.htmlWhitespaceSensitivity === 'ignore' ||
-				blockElements.includes(node.name as TagName))) ||
-		node.type === 'component' ||
-		node.type === 'fragment'
+		node &&
+		node.type === 'element' &&
+		opts.htmlWhitespaceSensitivity !== 'strict' &&
+		(opts.htmlWhitespaceSensitivity === 'ignore' || blockElements.includes(node.name as TagName))
 	);
 }
 
@@ -127,10 +124,6 @@ export function shouldHugStart(node: anyNode, opts: ParserOptions): boolean {
 		return false;
 	}
 
-	if (node.type === 'fragment') {
-		return false;
-	}
-
 	if (!isNodeWithChildren(node)) {
 		return false;
 	}

diff --git a/test/fixtures/basic/html-custom-elements/output.astro b/test/fixtures/basic/html-custom-elements/output.astro
@@ -1 +1 @@
-<custom-element></custom-element>
+<custom-element> </custom-element>
diff --git a/test/fixtures/markdown/embedded-in-markdown/input.md b/test/fixtures/markdown/embedded-in-markdown/input.md
@@ -59,9 +59,7 @@ For best results, you should only have one `<style>` tag per-Astro component. Th
       }
     </style>
   </head>
-  <body>
-    ...
-  </body>
+  <body> ...</body>
 </html>
 ```
 

diff --git a/test/fixtures/other/fragment/input.astro b/test/fixtures/other/fragment/input.astro
@@ -11,5 +11,9 @@
             Hello world!</>
           <Fragment>
             <span>lorem</span></Fragment>
+						<>Hello world!
+						</>
+          <Fragment><span>lorem</span>
+					</Fragment>
         </body>
 </html>
diff --git a/test/fixtures/other/fragment/output.astro b/test/fixtures/other/fragment/output.astro
@@ -7,9 +7,9 @@
     <title>Document</title>
   </head>
   <body>
-    <>Hello world!</>
-    <Fragment>
-      <span>lorem</span>
-    </Fragment>
+    <> Hello world!</>
+    <Fragment> <span>lorem</span></Fragment>
+    <>Hello world! </>
+    <Fragment><span>lorem</span> </Fragment>
   </body>
 </html>