]> source.dussan.org Git - sonarqube.git/commitdiff
SONAR-17724 Using incorrect ARIA role "tab" for Rules navigation (#7206)
authorDavid Cho-Lerat <117642976+david-cho-lerat-sonarsource@users.noreply.github.com>
Mon, 12 Dec 2022 10:24:24 +0000 (11:24 +0100)
committersonartech <sonartech@sonarsource.com>
Mon, 12 Dec 2022 20:04:02 +0000 (20:04 +0000)
server/sonar-web/src/main/js/apps/coding-rules/components/App.tsx
server/sonar-web/src/main/js/apps/coding-rules/components/__tests__/__snapshots__/App-test.tsx.snap

index e14c9daa65794ef9745c913a1a9e41aeba53a66f..a766fad57b2be4b2912812cded8e55bf7829f761 100644 (file)
@@ -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}
index f90b687a909016cba3e7b507aee3db2a7a180733..c21831ed804eb55f9db6d5548c283fc0fc900696 100644 (file)
@@ -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>