Add to .md and move up validRoles as a Set

jsx-eslint · Dec 3, 2021 · 77f6c87 · 77f6c87
1 parent f869a74
commit 77f6c87
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/docs/rules/aria-role.md b/docs/rules/aria-role.md
@@ -17,7 +17,7 @@ This rule takes one optional object argument of type object:
 }
 ```
 
-`allowedInvalidRules` is an optional string array of custom roles that should be allowed in addition to the ARIA spec.
+`allowedInvalidRules` is an optional string array of custom roles that should be allowed in addition to the ARIA spec, such as for cases when you [need to use a non-standard role](https://axesslab.com/text-splitting).
 
 For the `ignoreNonDOM` option, this determines if developer created components are checked.
 

diff --git a/src/rules/aria-role.js b/src/rules/aria-role.js
@@ -39,6 +39,7 @@ export default {
     const options = context.options[0] || {};
     const ignoreNonDOM = !!options.ignoreNonDOM;
     const allowedInvalidRoles = new Set(options.allowedInvalidRoles || [])
+    const validRoles = [...roles.keys()].filter((role) => roles.get(role).abstract === false);
 
     return ({
       JSXAttribute: (attribute) => {
@@ -63,7 +64,6 @@ export default {
         if (value === undefined || value === null) { return; }
 
         const values = String(value).split(' ');
-        const validRoles = [...roles.keys()].filter((role) => roles.get(role).abstract === false);
         const isValid = values.every((val) => allowedInvalidRoles.has(val) || validRoles.indexOf(val) > -1);
 
         if (isValid === true) { return; }