fix: never set custom element props as attributes

dummdidumm · dummdidumm · commit ebe803ff37a5 · 2024-07-26T20:52:24.000+02:00
diff --git a/packages/svelte/src/compiler/phases/3-transform/client/visitors/template.js b/packages/svelte/src/compiler/phases/3-transform/client/visitors/template.js
@@ -2152,6 +2152,7 @@ export const template_visitors = {
 				}
 
 				if (
+					!is_custom_element &&
 					attribute.name !== 'autofocus' &&
 					(attribute.value === true || is_text_attribute(attribute))
 				) {
diff --git a/packages/svelte/tests/runtime-legacy/samples/attribute-custom-element-inheritance/_config.js b/packages/svelte/tests/runtime-legacy/samples/attribute-custom-element-inheritance/_config.js
@@ -1,8 +1,7 @@
 import { test } from '../../test';
 
 export default test({
-	skip: true, // TODO: needs fixing
-
+	mode: ['client'],
 	html: `
 		<my-custom-inheritance-element>Hello World!</my-custom-inheritance-element>
 	`

Original file line number	Diff line number	Diff line change
`@@ -2152,6 +2152,7 @@ export const template_visitors = {`
`2152`	`2152`	`}`
`2153`	`2153`
`2154`	`2154`	`if (`
	`2155`	`+ !is_custom_element &&`
`2155`	`2156`	`attribute.name !== 'autofocus' &&`
`2156`	`2157`	`(attribute.value === true \|\| is_text_attribute(attribute))`
`2157`	`2158`	`) {`
Original file line number	Diff line number	Diff line change
`@@ -1,8 +1,7 @@`
`1`	`1`	`import { test } from '../../test';`
`2`	`2`
`3`	`3`	`export default test({`
`4`		`- skip: true, // TODO: needs fixing`
`5`		`-`
	`4`	`+ mode: ['client'],`
`6`	`5`	html: `
`7`	`6`	`<my-custom-inheritance-element>Hello World!</my-custom-inheritance-element>`
`8`	`7`	`