fix: silence autofocus a11y warning inside <dialog>

navorite · navorite · commit c02e9a5d9d52 · 2025-07-11T08:10:59.000+03:00
diff --git a/.changeset/cuddly-humans-end.md b/.changeset/cuddly-humans-end.md
@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: silence autofocus a11y warning inside `<dialog>`
diff --git a/packages/svelte/src/compiler/phases/2-analyze/visitors/shared/a11y.js b/packages/svelte/src/compiler/phases/2-analyze/visitors/shared/a11y.js
@@ -875,7 +875,7 @@ export function check_element(node, context) {
 		}
 
 		// no-autofocus
-		if (name === 'autofocus') {
+		if (name === 'autofocus' && node.name !== 'dialog' && !is_parent(context.path, ['dialog'])) {
 			w.a11y_autofocus(attribute);
 		}
 
diff --git a/packages/svelte/tests/validator/samples/a11y-no-autofocus/input.svelte b/packages/svelte/tests/validator/samples/a11y-no-autofocus/input.svelte
@@ -1 +1,6 @@
-<div autofocus></div>
+<div autofocus></div>
+<dialog autofocus>
+</dialog>
+<dialog>
+	<input autofocus>
+</dialog>

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +'svelte': patch
 +---
++
 +fix: silence autofocus a11y warning inside `<dialog>`
Original file line number	Diff line number	Diff line change
`@@ -875,7 +875,7 @@ export function check_element(node, context) {`
`875`	`875`	`}`
`876`	`876`
`877`	`877`	`// no-autofocus`
`878`		`- if (name === 'autofocus') {`
	`878`	`+ if (name === 'autofocus' && node.name !== 'dialog' && !is_parent(context.path, ['dialog'])) {`
`879`	`879`	`w.a11y_autofocus(attribute);`
`880`	`880`	`}`
`881`	`881`