Merge branch 'fix-lost-anchor-if-if-block-first' of https://github.co…

…m/stalkerg/svelte into stalkerg-fix-lost-anchor-if-if-block-first
sveltejs · Oct 20, 2018 · e5f5995 · e5f5995
2 parents 9e899db + 21ea87c
commit e5f5995
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 7 deletions.
diff --git a/src/compile/render-dom/wrappers/Fragment.ts b/src/compile/render-dom/wrappers/Fragment.ts
@@ -99,9 +99,7 @@ export default class FragmentWrapper {
 				this.nodes.unshift(wrapper);
 
 				link(lastChild, lastChild = wrapper);
-			}
-
-			else {
+			} else {
 				const Wrapper = wrappers[child.type];
 				if (!Wrapper) continue;
 
@@ -120,10 +118,6 @@ export default class FragmentWrapper {
 				if (!first.data) {
 					first.var = null;
 					this.nodes.shift();
-
-					if (this.nodes.length) {
-						link(null, this.nodes[0]);
-					}
 				}
 			}
 		}

diff --git a/test/runtime/samples/if-block-first/_config.js b/test/runtime/samples/if-block-first/_config.js
@@ -0,0 +1,12 @@
+export default {
+	data: {
+		visible: false
+	},
+
+	html: '<div><div>before me</div></div>',
+
+	test ( assert, component, target ) {
+		component.set({ visible: true });
+		assert.htmlEqual(target.innerHTML, '<div><div>i am visible</div><div>before me</div></div>' );
+	}
+};
diff --git a/test/runtime/samples/if-block-first/main.html b/test/runtime/samples/if-block-first/main.html
@@ -0,0 +1,6 @@
+<div>
+	{#if visible}
+		<div>i am visible</div>
+	{/if}
+	<div>before me</div>
+</div>