(fix) ignore whitespace sensitivity with Svelte components (#281) (#282)

mcuppi · web-flow · commit d91867579c8a · 2022-03-29T10:17:23.000+02:00
Fixes #281
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # prettier-plugin-svelte changelog
 
+## 2.6.1 (Unreleased)
+
+* (fix) respect whitespace "ignore" setting for components ([#281](https://github.com/sveltejs/prettier-plugin-svelte/issues/281)) 
+
 ## 2.6.0
 
 * (feat) Support `{@const}` tag ([#272](https://github.com/sveltejs/prettier-plugin-svelte/issues/272))
diff --git a/src/print/node-helpers.ts b/src/print/node-helpers.ts
@@ -396,6 +396,10 @@ export function shouldHugStart(
         return true;
     }
 
+    if (options.htmlWhitespaceSensitivity === 'ignore') {
+        return false;
+    }
+
     const firstChild = children[0];
     return !isTextNodeStartingWithWhitespace(firstChild);
 }
@@ -426,6 +430,10 @@ export function shouldHugEnd(
         return true;
     }
 
+    if (options.htmlWhitespaceSensitivity === 'ignore') {
+        return false;
+    }
+
     const lastChild = children[children.length - 1];
     return !isTextNodeEndingWithWhitespace(lastChild);
 }
diff --git a/test/formatting/samples/component-ignore-whitespace/input.html b/test/formatting/samples/component-ignore-whitespace/input.html
@@ -0,0 +1,5 @@
+<p><ExternalLink href="https://www.buymeacoffee.com">buy</ExternalLink>!</p>
+
+<p><ExternalLink href="https://www.buymeacoffee.com">buy me a</ExternalLink>!</p>
+
+<p><ExternalLink href="https://www.buymeacoffee.com">buy me a coffee</ExternalLink>!</p>
diff --git a/test/formatting/samples/component-ignore-whitespace/options.json b/test/formatting/samples/component-ignore-whitespace/options.json
@@ -0,0 +1,3 @@
+{
+    "htmlWhitespaceSensitivity": "ignore"
+}
diff --git a/test/formatting/samples/component-ignore-whitespace/output.html b/test/formatting/samples/component-ignore-whitespace/output.html
@@ -0,0 +1,11 @@
+<p><ExternalLink href="https://www.buymeacoffee.com">buy</ExternalLink>!</p>
+
+<p>
+    <ExternalLink href="https://www.buymeacoffee.com">buy me a</ExternalLink>!
+</p>
+
+<p>
+    <ExternalLink href="https://www.buymeacoffee.com">
+        buy me a coffee
+    </ExternalLink>!
+</p>

Original file line number	Diff line number	Diff line change
`@@ -396,6 +396,10 @@ export function shouldHugStart(`
`396`	`396`	`return true;`
`397`	`397`	`}`
`398`	`398`
	`399`	`+ if (options.htmlWhitespaceSensitivity === 'ignore') {`
	`400`	`+ return false;`
	`401`	`+ }`
	`402`	`+`
`399`	`403`	`const firstChild = children[0];`
`400`	`404`	`return !isTextNodeStartingWithWhitespace(firstChild);`
`401`	`405`	`}`
`@@ -426,6 +430,10 @@ export function shouldHugEnd(`
`426`	`430`	`return true;`
`427`	`431`	`}`
`428`	`432`
	`433`	`+ if (options.htmlWhitespaceSensitivity === 'ignore') {`
	`434`	`+ return false;`
	`435`	`+ }`
	`436`	`+`
`429`	`437`	`const lastChild = children[children.length - 1];`
`430`	`438`	`return !isTextNodeEndingWithWhitespace(lastChild);`
`431`	`439`	`}`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+{`
	`2`	`+ "htmlWhitespaceSensitivity": "ignore"`
	`3`	`+}`