From 1cfed0bf2980567787edd1d813f6ce50eaf3f033 Mon Sep 17 00:00:00 2001
From: Simon H <5968653+dummdidumm@users.noreply.github.com>
Date: Wed, 28 Jun 2023 12:29:44 +0200
Subject: [PATCH] fix: relax a11y "no redundant role" rule for li, ul, ol
 (#8867)

fixes #8854
---
 .changeset/pretty-tools-whisper.md            |  5 +++
 .../src/compiler/compile/nodes/Element.js     |  6 +++-
 .../a11y-no-redundant-roles/input.svelte      | 12 +++----
 .../a11y-no-redundant-roles/warnings.json     | 36 -------------------
 4 files changed, 16 insertions(+), 43 deletions(-)
 create mode 100644 .changeset/pretty-tools-whisper.md

diff --git a/.changeset/pretty-tools-whisper.md b/.changeset/pretty-tools-whisper.md
new file mode 100644
index 0000000000..8af633242f
--- /dev/null
+++ b/.changeset/pretty-tools-whisper.md
@@ -0,0 +1,5 @@
+---
+'svelte': patch
+---
+
+fix: relax a11y "no redundant role" rule for li, ul, ol
diff --git a/packages/svelte/src/compiler/compile/nodes/Element.js b/packages/svelte/src/compiler/compile/nodes/Element.js
index 3b18ce0e94..ecefb0c34e 100644
--- a/packages/svelte/src/compiler/compile/nodes/Element.js
+++ b/packages/svelte/src/compiler/compile/nodes/Element.js
@@ -694,7 +694,11 @@ export default class Element extends Node {
 									);
 								}
 								// no-redundant-roles
-								if (current_role === get_implicit_role(this.name, attribute_map)) {
+								if (
+									current_role === get_implicit_role(this.name, attribute_map) &&
+									// <ul role="list"> is ok because CSS list-style:none removes the semantics and this is a way to bring them back
+									!['ul', 'ol', 'li'].includes(this.name)
+								) {
 									component.warn(
 										attribute,
 										compiler_warnings.a11y_no_redundant_roles(current_role)
diff --git a/packages/svelte/test/validator/samples/a11y-no-redundant-roles/input.svelte b/packages/svelte/test/validator/samples/a11y-no-redundant-roles/input.svelte
index 537d5c0fd3..e5c987d691 100644
--- a/packages/svelte/test/validator/samples/a11y-no-redundant-roles/input.svelte
+++ b/packages/svelte/test/validator/samples/a11y-no-redundant-roles/input.svelte
@@ -19,12 +19,12 @@
 <h5 role="heading">heading</h5>
 <h6 role="heading">heading</h6>
 <hr role="separator" />
-<li role="listitem" />
+<!-- <li role="listitem" /> allowed since CSS list-style none removes semantic meaning and role brings it back -->
 <link role="link" />
-<main role="main"></main>
+<main role="main" />
 <menu role="list" />
 <nav role="navigation" />
-<ol role="list" />
+<!-- <ol role="list" /> allowed, see comment above -->
 <optgroup role="group" />
 <option role="option" />
 <output role="status" />
@@ -37,11 +37,11 @@
 <tfoot role="rowgroup" />
 <thead role="rowgroup" />
 <tr role="row" />
-<ul role="list" />
+<!--<ul role="list" /> allowed, see comment above -->
 
 <!-- Tested header/footer not nested in section/article -->
-<header role="banner"></header>
-<footer role="contentinfo"></footer>
+<header role="banner" />
+<footer role="contentinfo" />
 
 <!-- Allowed -->
 <!-- svelte-ignore a11y-no-noninteractive-element-to-interactive-role -->
diff --git a/packages/svelte/test/validator/samples/a11y-no-redundant-roles/warnings.json b/packages/svelte/test/validator/samples/a11y-no-redundant-roles/warnings.json
index 4fcd60385e..12cdc6f8d5 100644
--- a/packages/svelte/test/validator/samples/a11y-no-redundant-roles/warnings.json
+++ b/packages/svelte/test/validator/samples/a11y-no-redundant-roles/warnings.json
@@ -251,18 +251,6 @@
 			"line": 21
 		}
 	},
-	{
-		"code": "a11y-no-redundant-roles",
-		"end": {
-			"column": 19,
-			"line": 22
-		},
-		"message": "A11y: Redundant role 'listitem'",
-		"start": {
-			"column": 4,
-			"line": 22
-		}
-	},
 	{
 		"code": "a11y-no-redundant-roles",
 		"end": {
@@ -311,18 +299,6 @@
 			"line": 26
 		}
 	},
-	{
-		"code": "a11y-no-redundant-roles",
-		"end": {
-			"column": 15,
-			"line": 27
-		},
-		"message": "A11y: Redundant role 'list'",
-		"start": {
-			"column": 4,
-			"line": 27
-		}
-	},
 	{
 		"code": "a11y-no-redundant-roles",
 		"end": {
@@ -467,18 +443,6 @@
 			"line": 39
 		}
 	},
-	{
-		"code": "a11y-no-redundant-roles",
-		"end": {
-			"column": 15,
-			"line": 40
-		},
-		"message": "A11y: Redundant role 'list'",
-		"start": {
-			"column": 4,
-			"line": 40
-		}
-	},
 	{
 		"code": "a11y-no-redundant-roles",
 		"end": {