From e1b55f0fc3c9b2ceec58c6e5ff8ccf16e04a0c9f Mon Sep 17 00:00:00 2001
From: Tan Li Hau <tanhauhau@users.noreply.github.com>
Date: Thu, 25 Jun 2020 06:17:21 +0800
Subject: [PATCH 1/5] add further cases where {@html} uses an anchor (#5061)

---
 CHANGELOG.md                                  |  1 +
 .../render_dom/wrappers/RawMustacheTag.ts     |  2 +-
 .../raw-mustache-as-root/RawMustache.svelte   |  5 +++
 .../samples/raw-mustache-as-root/_config.js   | 33 +++++++++++++++++++
 .../samples/raw-mustache-as-root/main.svelte  | 17 ++++++++++
 .../Component.svelte                          |  2 ++
 .../raw-mustache-before-element/_config.js    | 32 +++++++++++++++++-
 .../raw-mustache-before-element/main.svelte   | 18 +++++++++-
 .../raw-mustache-inside-slot/_config.js       |  3 ++
 .../raw-mustache-inside-slot/main.svelte      |  1 +
 10 files changed, 111 insertions(+), 3 deletions(-)
 create mode 100644 test/runtime/samples/raw-mustache-as-root/RawMustache.svelte
 create mode 100644 test/runtime/samples/raw-mustache-as-root/_config.js
 create mode 100644 test/runtime/samples/raw-mustache-as-root/main.svelte
 create mode 100644 test/runtime/samples/raw-mustache-before-element/Component.svelte
 create mode 100644 test/runtime/samples/raw-mustache-inside-slot/_config.js
 create mode 100644 test/runtime/samples/raw-mustache-inside-slot/main.svelte

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 4ebb458476..9650d9e0d6 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -2,6 +2,7 @@
 
 ## Unreleased
 
+* Fix placement of `{@html}` when used at the root of a slot or the root of a component ([#5012](https://github.com/sveltejs/svelte/issues/5012))
 * Fix handling of `import`ed value that is used as a store and is also mutated ([#5019](https://github.com/sveltejs/svelte/issues/5019))
 * Do not display `a11y-missing-content` warning on elements with `contenteditable` bindings ([#5020](https://github.com/sveltejs/svelte/issues/5020))
 * Fix handling of `this` in inline function expressions in the template ([#5033](https://github.com/sveltejs/svelte/issues/5033))
diff --git a/src/compiler/compile/render_dom/wrappers/RawMustacheTag.ts b/src/compiler/compile/render_dom/wrappers/RawMustacheTag.ts
index 1e6213cf02..a5367b207d 100644
--- a/src/compiler/compile/render_dom/wrappers/RawMustacheTag.ts
+++ b/src/compiler/compile/render_dom/wrappers/RawMustacheTag.ts
@@ -39,7 +39,7 @@ export default class RawMustacheTagWrapper extends Tag {
 		}
 
 		else {
-			const needs_anchor = in_head || (this.next && !this.next.is_dom_node());
+			const needs_anchor = in_head || (this.next ? !this.next.is_dom_node() : (!this.parent || !this.parent.is_dom_node()));
 
 			const html_tag = block.get_unique_name('html_tag');
 			const html_anchor = needs_anchor && block.get_unique_name('html_anchor');
diff --git a/test/runtime/samples/raw-mustache-as-root/RawMustache.svelte b/test/runtime/samples/raw-mustache-as-root/RawMustache.svelte
new file mode 100644
index 0000000000..d94954f49b
--- /dev/null
+++ b/test/runtime/samples/raw-mustache-as-root/RawMustache.svelte
@@ -0,0 +1,5 @@
+<script>
+	export let content;
+</script>
+
+{@html content}
\ No newline at end of file
diff --git a/test/runtime/samples/raw-mustache-as-root/_config.js b/test/runtime/samples/raw-mustache-as-root/_config.js
new file mode 100644
index 0000000000..c87971a7f0
--- /dev/null
+++ b/test/runtime/samples/raw-mustache-as-root/_config.js
@@ -0,0 +1,33 @@
+export default {
+	html: `
+		<button>Switch</button>
+		<p>Another first line</p>
+		<p>This line should be last.</p>
+	`,
+	async test({ assert, target, window }) {
+		const btn = target.querySelector("button");
+		const clickEvent = new window.MouseEvent("click");
+
+		await btn.dispatchEvent(clickEvent);
+
+		assert.htmlEqual(
+			target.innerHTML,
+			`
+				<button>Switch</button>
+				<p>First line</p>
+				<p>This line should be last.</p>
+			`
+		);
+
+		await btn.dispatchEvent(clickEvent);
+
+		assert.htmlEqual(
+			target.innerHTML,
+			`
+				<button>Switch</button>
+				<p>Another first line</p>
+				<p>This line should be last.</p>
+			`
+		);
+	},
+};
diff --git a/test/runtime/samples/raw-mustache-as-root/main.svelte b/test/runtime/samples/raw-mustache-as-root/main.svelte
new file mode 100644
index 0000000000..7ccce0cd9b
--- /dev/null
+++ b/test/runtime/samples/raw-mustache-as-root/main.svelte
@@ -0,0 +1,17 @@
+<script>
+	import RawMustache from './RawMustache.svelte';
+
+	let content1 = `<p>First line</p>`;
+	let content2 = `<p>Another first line</p>`;
+
+	let show = false;
+	$: content = show ? content1 : content2;
+</script>
+
+<button on:click={() => show = !show}>
+	Switch
+</button>
+
+<RawMustache {content} />
+
+<p>This line should be last.</p>
\ No newline at end of file
diff --git a/test/runtime/samples/raw-mustache-before-element/Component.svelte b/test/runtime/samples/raw-mustache-before-element/Component.svelte
new file mode 100644
index 0000000000..fcabccae48
--- /dev/null
+++ b/test/runtime/samples/raw-mustache-before-element/Component.svelte
@@ -0,0 +1,2 @@
+<slot />
+<p>This line should be last.</p>
\ No newline at end of file
diff --git a/test/runtime/samples/raw-mustache-before-element/_config.js b/test/runtime/samples/raw-mustache-before-element/_config.js
index 61288cbb52..c87971a7f0 100644
--- a/test/runtime/samples/raw-mustache-before-element/_config.js
+++ b/test/runtime/samples/raw-mustache-before-element/_config.js
@@ -1,3 +1,33 @@
 export default {
-	html: `<p>x<span>baz</span></p>`
+	html: `
+		<button>Switch</button>
+		<p>Another first line</p>
+		<p>This line should be last.</p>
+	`,
+	async test({ assert, target, window }) {
+		const btn = target.querySelector("button");
+		const clickEvent = new window.MouseEvent("click");
+
+		await btn.dispatchEvent(clickEvent);
+
+		assert.htmlEqual(
+			target.innerHTML,
+			`
+				<button>Switch</button>
+				<p>First line</p>
+				<p>This line should be last.</p>
+			`
+		);
+
+		await btn.dispatchEvent(clickEvent);
+
+		assert.htmlEqual(
+			target.innerHTML,
+			`
+				<button>Switch</button>
+				<p>Another first line</p>
+				<p>This line should be last.</p>
+			`
+		);
+	},
 };
diff --git a/test/runtime/samples/raw-mustache-before-element/main.svelte b/test/runtime/samples/raw-mustache-before-element/main.svelte
index 69c1d0107d..a022a68e95 100644
--- a/test/runtime/samples/raw-mustache-before-element/main.svelte
+++ b/test/runtime/samples/raw-mustache-before-element/main.svelte
@@ -1 +1,17 @@
-<p>{@html 'x'}<span>baz</span></p>
\ No newline at end of file
+<script>
+	import Component from './Component.svelte';
+
+	let content1 = `<p>First line</p>`;
+	let content2 = `<p>Another first line</p>`
+
+	let show = false;
+	$: content = show ? content1 : content2;
+</script>
+
+<button on:click={() => show = !show}>
+	Switch
+</button>
+
+<Component>
+	{@html content}
+</Component>
\ No newline at end of file
diff --git a/test/runtime/samples/raw-mustache-inside-slot/_config.js b/test/runtime/samples/raw-mustache-inside-slot/_config.js
new file mode 100644
index 0000000000..61288cbb52
--- /dev/null
+++ b/test/runtime/samples/raw-mustache-inside-slot/_config.js
@@ -0,0 +1,3 @@
+export default {
+	html: `<p>x<span>baz</span></p>`
+};
diff --git a/test/runtime/samples/raw-mustache-inside-slot/main.svelte b/test/runtime/samples/raw-mustache-inside-slot/main.svelte
new file mode 100644
index 0000000000..69c1d0107d
--- /dev/null
+++ b/test/runtime/samples/raw-mustache-inside-slot/main.svelte
@@ -0,0 +1 @@
+<p>{@html 'x'}<span>baz</span></p>
\ No newline at end of file

From 0520a10dc0b93b754dc42a2f233efdfbd25fa162 Mon Sep 17 00:00:00 2001
From: Bassam Ismail <skippednote@gmail.com>
Date: Thu, 25 Jun 2020 06:13:03 +0530
Subject: [PATCH 2/5] site: check if mapbox key is set before generating the
 production build (#5057)

---
 site/rollup.config.js | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/site/rollup.config.js b/site/rollup.config.js
index 9cb963a87d..d4efb4eb29 100644
--- a/site/rollup.config.js
+++ b/site/rollup.config.js
@@ -13,6 +13,10 @@ const mode = process.env.NODE_ENV;
 const dev = mode === 'development';
 const legacy = !!process.env.SAPPER_LEGACY_BUILD;
 
+if (!dev && !process.env.MAPBOX_ACCESS_TOKEN) {
+	throw new Error('MAPBOX_ACCESS_TOKEN is missing. Please add the token in the .env file before generating the production build.');
+}
+
 const onwarn = (warning, onwarn) => (warning.code === 'CIRCULAR_DEPENDENCY' && /[/\\]@sapper[/\\]/.test(warning.message)) || onwarn(warning);
 const dedupe = importee => importee === 'svelte' || importee.startsWith('svelte/');
 

From 1a71e0407905a8f023794fedd752c8e0eec06d2f Mon Sep 17 00:00:00 2001
From: Tan Li Hau <tanhauhau@users.noreply.github.com>
Date: Thu, 25 Jun 2020 20:33:30 +0800
Subject: [PATCH 3/5] fix bind:this, skip checking before adding to
 binding_callbacks (#5072)

---
 CHANGELOG.md                                  |  1 +
 .../render_dom/wrappers/shared/bind_this.ts   |  3 +-
 .../_config.js                                | 53 +++++++++++++++++++
 .../main.svelte                               | 17 ++++++
 4 files changed, 72 insertions(+), 2 deletions(-)
 create mode 100644 test/runtime/samples/binding-this-each-block-property-2/_config.js
 create mode 100644 test/runtime/samples/binding-this-each-block-property-2/main.svelte

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 9650d9e0d6..8457cd120b 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -7,6 +7,7 @@
 * Do not display `a11y-missing-content` warning on elements with `contenteditable` bindings ([#5020](https://github.com/sveltejs/svelte/issues/5020))
 * Fix handling of `this` in inline function expressions in the template ([#5033](https://github.com/sveltejs/svelte/issues/5033))
 * Update `<select>` with one-way `value` binding when the available `<option>`s change ([#5051](https://github.com/sveltejs/svelte/issues/5051))
+* Fix contextual `bind:this` inside `{#each}` block ([#5067](https://github.com/sveltejs/svelte/issues/5067))
 
 ## 3.23.2
 
diff --git a/src/compiler/compile/render_dom/wrappers/shared/bind_this.ts b/src/compiler/compile/render_dom/wrappers/shared/bind_this.ts
index 078eb1839d..c3aa04960e 100644
--- a/src/compiler/compile/render_dom/wrappers/shared/bind_this.ts
+++ b/src/compiler/compile/render_dom/wrappers/shared/bind_this.ts
@@ -11,7 +11,7 @@ export default function bind_this(component: Component, block: Block, binding: B
 	block.renderer.add_to_context(fn.name);
 	const callee = block.renderer.reference(fn.name);
 
-	const { contextual_dependencies, mutation, lhs } = binding.handler;
+	const { contextual_dependencies, mutation } = binding.handler;
 	const dependencies = binding.get_dependencies();
 
 	const body = b`
@@ -29,7 +29,6 @@ export default function bind_this(component: Component, block: Block, binding: B
 		}));
 		component.partly_hoisted.push(b`
 			function ${fn}($$value, ${params}) {
-				if (${lhs} === $$value) return;
 				@binding_callbacks[$$value ? 'unshift' : 'push'](() => {
 					${body}
 				});
diff --git a/test/runtime/samples/binding-this-each-block-property-2/_config.js b/test/runtime/samples/binding-this-each-block-property-2/_config.js
new file mode 100644
index 0000000000..eebd754fca
--- /dev/null
+++ b/test/runtime/samples/binding-this-each-block-property-2/_config.js
@@ -0,0 +1,53 @@
+let calls = [];
+function callback(refs) {
+	calls.push(refs.map(({ ref }) => ({ ref })));
+}
+export default {
+	html: ``,
+	props: {
+		callback,
+	},
+	after_test() {
+		calls = [];
+	},
+	async test({ assert, component, target }) {
+		assert.equal(calls.length, 1);
+		assert.equal(calls[0].length, 0);
+
+		await component.addItem();
+
+		let divs = target.querySelectorAll("div");
+
+		assert.equal(calls.length, 3);
+		assert.equal(calls[1].length, 1);
+		assert.equal(calls[1][0].ref, null);
+		assert.equal(calls[2].length, 1);
+		assert.equal(calls[2][0].ref, divs[0]);
+
+		await component.addItem();
+
+		divs = target.querySelectorAll("div");
+
+		assert.equal(calls.length, 5);
+		assert.equal(calls[3].length, 2);
+		assert.equal(calls[3][0].ref, divs[0]);
+		assert.equal(calls[3][1].ref, null);
+		assert.equal(calls[4].length, 2);
+		assert.equal(calls[4][0].ref, divs[0]);
+		assert.equal(calls[4][1].ref, divs[1]);
+
+		await component.addItem();
+
+		divs = target.querySelectorAll("div");
+
+		assert.equal(calls.length, 7);
+		assert.equal(calls[5].length, 3);
+		assert.equal(calls[5][0].ref, divs[0]);
+		assert.equal(calls[5][1].ref, divs[1]);
+		assert.equal(calls[5][2].ref, null);
+		assert.equal(calls[6].length, 3);
+		assert.equal(calls[6][0].ref, divs[0]);
+		assert.equal(calls[6][1].ref, divs[1]);
+		assert.equal(calls[6][2].ref, divs[2]);
+	},
+};
diff --git a/test/runtime/samples/binding-this-each-block-property-2/main.svelte b/test/runtime/samples/binding-this-each-block-property-2/main.svelte
new file mode 100644
index 0000000000..bc1efe725c
--- /dev/null
+++ b/test/runtime/samples/binding-this-each-block-property-2/main.svelte
@@ -0,0 +1,17 @@
+<script>
+	import { tick } from 'svelte';
+	let refs = [];
+
+	export function addItem() {
+		refs = refs.concat({ ref: null });
+		return tick();
+	}
+
+	export let callback;
+
+	$: callback(refs);
+</script>
+
+{#each refs as xxx}
+	<div bind:this={xxx.ref} />
+{/each}
\ No newline at end of file

From 46fe407478995e223fbce9779d0852409d277721 Mon Sep 17 00:00:00 2001
From: Bassam Ismail <skippednote@gmail.com>
Date: Thu, 25 Jun 2020 20:35:50 +0530
Subject: [PATCH 4/5] make tweened options optional in types (#5068)

---
 src/runtime/motion/tweened.ts | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/runtime/motion/tweened.ts b/src/runtime/motion/tweened.ts
index c802604c0e..83434cdff6 100644
--- a/src/runtime/motion/tweened.ts
+++ b/src/runtime/motion/tweened.ts
@@ -64,9 +64,9 @@ interface Options<T> {
 type Updater<T> = (target_value: T, value: T) => T;
 
 interface Tweened<T> extends Readable<T> {
-	set(value: T, opts: Options<T>): Promise<void>;
+	set(value: T, opts?: Options<T>): Promise<void>;
 
-	update(updater: Updater<T>, opts: Options<T>): Promise<void>;
+	update(updater: Updater<T>, opts?: Options<T>): Promise<void>;
 }
 
 export function tweened<T>(value?: T, defaults: Options<T> = {}): Tweened<T> {
@@ -75,7 +75,7 @@ export function tweened<T>(value?: T, defaults: Options<T> = {}): Tweened<T> {
 	let task: Task;
 	let target_value = value;
 
-	function set(new_value: T, opts: Options<T>) {
+	function set(new_value: T, opts?: Options<T>) {
 		if (value == null) {
 			store.set(value = new_value);
 			return Promise.resolve();
@@ -98,7 +98,7 @@ export function tweened<T>(value?: T, defaults: Options<T> = {}): Tweened<T> {
 				previous_task.abort();
 				previous_task = null;
 			}
-			
+
 			store.set(value = target_value);
 			return Promise.resolve();
 		}
@@ -137,7 +137,7 @@ export function tweened<T>(value?: T, defaults: Options<T> = {}): Tweened<T> {
 
 	return {
 		set,
-		update: (fn, opts: Options<T>) => set(fn(target_value, value), opts),
+		update: (fn, opts?: Options<T>) => set(fn(target_value, value), opts),
 		subscribe: store.subscribe
 	};
 }

From d3ee85e5fe728f6de3f286b8877a2aa45e80c1e1 Mon Sep 17 00:00:00 2001
From: Conduitry <git@chor.date>
Date: Thu, 25 Jun 2020 11:06:52 -0400
Subject: [PATCH 5/5] update changelog

---
 CHANGELOG.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 8457cd120b..d760b0e8dd 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -7,6 +7,7 @@
 * Do not display `a11y-missing-content` warning on elements with `contenteditable` bindings ([#5020](https://github.com/sveltejs/svelte/issues/5020))
 * Fix handling of `this` in inline function expressions in the template ([#5033](https://github.com/sveltejs/svelte/issues/5033))
 * Update `<select>` with one-way `value` binding when the available `<option>`s change ([#5051](https://github.com/sveltejs/svelte/issues/5051))
+* Fix published `tweened` types so the `.set()` and `.update()` options are optional ([#5062](https://github.com/sveltejs/svelte/issues/5062))
 * Fix contextual `bind:this` inside `{#each}` block ([#5067](https://github.com/sveltejs/svelte/issues/5067))
 
 ## 3.23.2