From e051863b7be7bdfe2f5c59ce921788b3d7ca0c15 Mon Sep 17 00:00:00 2001
From: David Cho-Lerat
 <117642976+david-cho-lerat-sonarsource@users.noreply.github.com>
Date: Mon, 12 Dec 2022 11:24:24 +0100
Subject: [PATCH] SONAR-17724 Using incorrect ARIA role "tab" for Rules
 navigation (#7206)

---
 .../src/main/js/apps/coding-rules/components/App.tsx   |  4 ++--
 .../__tests__/__snapshots__/App-test.tsx.snap          | 10 +++-------
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/server/sonar-web/src/main/js/apps/coding-rules/components/App.tsx b/server/sonar-web/src/main/js/apps/coding-rules/components/App.tsx
index e14c9daa657..a766fad57b2 100644
--- a/server/sonar-web/src/main/js/apps/coding-rules/components/App.tsx
+++ b/server/sonar-web/src/main/js/apps/coding-rules/components/App.tsx
@@ -647,7 +647,7 @@ export class App extends React.PureComponent<Props, State> {
                 />
               ) : (
                 <>
-                  <div role="tab">
+                  <section>
                     {rules.map((rule) => (
                       <RuleListItem
                         activation={this.getRuleActivation(rule.key)}
@@ -662,7 +662,7 @@ export class App extends React.PureComponent<Props, State> {
                         selectedProfile={this.getSelectedProfile()}
                       />
                     ))}
-                  </div>
+                  </section>
                   {paging !== undefined && (
                     <ListFooter
                       count={rules.length}
diff --git a/server/sonar-web/src/main/js/apps/coding-rules/components/__tests__/__snapshots__/App-test.tsx.snap b/server/sonar-web/src/main/js/apps/coding-rules/components/__tests__/__snapshots__/App-test.tsx.snap
index f90b687a909..c21831ed804 100644
--- a/server/sonar-web/src/main/js/apps/coding-rules/components/__tests__/__snapshots__/App-test.tsx.snap
+++ b/server/sonar-web/src/main/js/apps/coding-rules/components/__tests__/__snapshots__/App-test.tsx.snap
@@ -267,9 +267,7 @@ exports[`should render correctly: loaded 1`] = `
       <div
         className="layout-page-main-inner"
       >
-        <div
-          role="tab"
-        >
+        <section>
           <RuleListItem
             isLoggedIn={true}
             key="javascript:S1067"
@@ -324,7 +322,7 @@ exports[`should render correctly: loaded 1`] = `
             }
             selected={false}
           />
-        </div>
+        </section>
         <ListFooter
           count={2}
           loadMore={[Function]}
@@ -389,9 +387,7 @@ exports[`should render correctly: loading 1`] = `
       <div
         className="layout-page-main-inner"
       >
-        <div
-          role="tab"
-        />
+        <section />
       </div>
     </div>
   </div>
-- 
2.39.5