fix self closing tag problems

Author: manuel3108

packages/svelte/src/compiler/print/index.js

@@ -714,35 +714,34 @@ const svelte_visitors = {
 			context.visit(attribute);
 		}
 
-		if (node.fragment) {
+		if (node.fragment && node.fragment.nodes.length > 0) {
 			context.write('>');
 			block(context, node.fragment, true);
 			context.write(`</svelte:boundary>`);
 		} else {
-			context.write('/>');
+			context.write(' />');
 		}
 	},
 
 	SvelteComponent(node, context) {
 		context.write('<svelte:component');
 
-		context.write('this={');
+		context.write(' this={');
 		context.visit(node.expression);
-		context.write('} ');
+		context.write('}');
 
 		for (const attribute of node.attributes) {
 			// TODO handle multiline
 			context.write(' ');
 			context.visit(attribute);
 		}
 
-		// TODO handling of self-closing does not seem to work
-		if (node.fragment) {
+		if (node.fragment && node.fragment.nodes.length > 0) {
 			context.write('>');
 			block(context, node.fragment, true);
 			context.write(`</svelte:component>`);
 		} else {
-			context.write('/>');
+			context.write(' />');
 		}
 	},
 
@@ -755,13 +754,12 @@ const svelte_visitors = {
 			context.visit(attribute);
 		}
 
-		// TODO handling of self-closing does not seem to work
-		if (node.fragment) {
+		if (node.fragment && node.fragment.nodes.length > 0) {
 			context.write('>');
 			block(context, node.fragment, true);
 			context.write(`</svelte:document>`);
 		} else {
-			context.write('/>');
+			context.write(' />');
 		}
 	},
 
@@ -779,12 +777,12 @@ const svelte_visitors = {
 		}
 
 		// TODO new line handling not working?
-		if (node.fragment) {
+		if (node.fragment && node.fragment.nodes.length > 0) {
 			context.write('>');
 			block(context, node.fragment, true);
 			context.write(`</svelte:element>`);
 		} else {
-			context.write('/>');
+			context.write(' />');
 		}
 	},
 
@@ -797,12 +795,12 @@ const svelte_visitors = {
 			context.visit(attribute);
 		}
 
-		if (node.fragment) {
+		if (node.fragment && node.fragment.nodes.length > 0) {
 			context.write('>');
 			block(context, node.fragment, true);
 			context.write(`</svelte:fragment>`);
 		} else {
-			context.write('/>');
+			context.write(' />');
 		}
 	},
 
@@ -815,12 +813,12 @@ const svelte_visitors = {
 			context.visit(attribute);
 		}
 
-		if (node.fragment) {
+		if (node.fragment && node.fragment.nodes.length > 0) {
 			context.write('>');
 			block(context, node.fragment, true);
 			context.write(`</svelte:head>`);
 		} else {
-			context.write('/>');
+			context.write(' />');
 		}
 	},
 
@@ -833,13 +831,12 @@ const svelte_visitors = {
 			context.visit(attribute);
 		}
 
-		// TODO handling of self-closing does not seem to work
-		if (node.fragment) {
+		if (node.fragment && node.fragment.nodes.length > 0) {
 			context.write('>');
 			block(context, node.fragment, true);
 			context.write(`</svelte:self>`);
 		} else {
-			context.write('/>');
+			context.write(' />');
 		}
 	},
 
@@ -852,13 +849,12 @@ const svelte_visitors = {
 			context.visit(attribute);
 		}
 
-		// TODO handling of self-closing does not seem to work
-		if (node.fragment) {
+		if (node.fragment && node.fragment.nodes.length > 0) {
 			context.write('>');
 			block(context, node.fragment, true);
 			context.write(`</svelte:window>`);
 		} else {
-			context.write('/>');
+			context.write(' />');
 		}
 	},
 

packages/svelte/tests/print/samples/if-block/output.svelte

@@ -1,5 +1,5 @@
 {#if porridge.temperature > 100}
-	<p>too hot1!</p>
+	<p>too hot!</p>
 {:else if 80 > porridge.temperature}
 	<p>too cold!</p>
 {:else}

packages/svelte/tests/print/samples/svelte-component/output.svelte

@@ -1 +1 @@
-<svelte:componentthis={MyComponent} ></svelte:component>
+<svelte:component this={MyComponent} />

packages/svelte/tests/print/samples/svelte-document/output.svelte

@@ -1 +1 @@
-<svelte:document onvisibilitychange={handleVisibilityChange} use:someAction></svelte:document>
+<svelte:document onvisibilitychange={handleVisibilityChange} use:someAction />

packages/svelte/tests/print/samples/svelte-self/output.svelte

@@ -4,7 +4,7 @@
 
 {#if count > 0}
 	<p>counting down... {count}</p>
-	<svelte:self count={count - 1}></svelte:self>
+	<svelte:self count={count - 1} />
 {:else}
 	<p>lift-off!</p>
 {/if}

packages/svelte/tests/print/samples/svelte-window/output.svelte

@@ -4,4 +4,4 @@
 	}
 </script>
 
-<svelte:window onkeydown={handleKeydown}></svelte:window>
+<svelte:window onkeydown={handleKeydown} />