Merge pull request #821 from classtranscribe/hd/bump-a11y-rules

Bump a11y rules
classtranscribe · Jun 11, 2024 · 2255c8a · 2255c8a
2 parents e354828 + 8229922
commit 2255c8a
Show file tree

Hide file tree

Showing 3 changed files with 477 additions and 46 deletions.
diff --git a/package.json b/package.json
@@ -155,7 +155,7 @@
     "eslint-plugin-flowtype": "^5.2.0",
     "eslint-plugin-import": "^2.22.1",
     "eslint-plugin-jest": "^24.1.0",
-    "eslint-plugin-jsx-a11y": "^6.3.1",
+    "eslint-plugin-jsx-a11y": "^6.8.0",
     "eslint-plugin-react": "^7.21.5",
     "eslint-plugin-react-hooks": "^4.2.0",
     "eslint-plugin-testing-library": "^3.9.2",

diff --git a/src/screens/Glossary/components/GlossaryTable/index.js b/src/screens/Glossary/components/GlossaryTable/index.js
@@ -247,8 +247,12 @@ const GlossaryTable = props => {
                   DOMAIN
                 </button>
               </th> */}
+
               <th>LIKE</th>
+              { /* Not sure why this triggers the input label accessibility role, might be a library bug since it's just a space */}
+              { /* eslint-disable jsx-a11y/control-has-associated-label */ }
               <th>&nbsp;</th>
+              { /* eslint-enable jsx-a11y/control-has-associated-label */ }
             </tr>
           </thead>
           <tbody>