Merge pull request #1910 from sveltejs/gh-1908

don't lump in `bind:this` with everything else

src/compile/render-dom/wrappers/Element/index.ts

@@ -403,7 +403,9 @@ export default class ElementWrapper extends Wrapper {
 		const groups = events
 			.map(event => ({
 				events: event.eventNames,
-				bindings: mungedBindings.filter(binding => event.filter(this.node, binding.name))
+				bindings: mungedBindings
+					.filter(binding => binding.name !== 'this')
+					.filter(binding => event.filter(this.node, binding.name))
 			}))
 			.filter(group => group.bindings.length);
 

test/runtime/samples/binding-this-and-value/_config.js

@@ -0,0 +1,24 @@
+export default {
+	html: `
+		<input>
+		<p>value: initial</p>
+	`,
+
+	ssrHtml: `
+		<input value="initial">
+		<p>value: initial</p>
+	`,
+
+	async test({ assert, target, window }) {
+		const input = target.querySelector('input');
+		const event = new window.Event('input');
+
+		input.value = 'changed';
+		await input.dispatchEvent(event);
+
+		assert.htmlEqual(target.innerHTML, `
+			<input>
+			<p>value: changed</p>
+		`);
+	}
+};

test/runtime/samples/binding-this-and-value/main.html

@@ -0,0 +1,7 @@
+<script>
+	let node;
+	export let value = 'initial';
+</script>
+
+<input bind:this={node} bind:value>
+<p>value: {value}</p>