[Rule Request]: Prohibiting use of aria-hidden and role='presentation' on focusable elements.

[mauryapari]

• Closes [Rule Request] Rule for prohibiting use of aria-hidden or role=presentation on focusable elements #85
• Added rule for prohibiting use of aria-hidden and role='presentation' on focusable elements.
• Add checks to prohibit use of aria-hidden and role='presentation' on parents of focusable elements.

[mauryapari]

@kddnewton , @vhoyer. Could you please reiview this PR? Any feedback you provide would be greatly appreciated.

[vhoyer]

Hey, could you add a new line at the end of the files that do not have them? github adds a little 🚫 at the end of the file when that's the case

[vhoyer]

code wise it seems fine, I just don't like nested ifs if I can avoid it, but not a real problem

[vhoyer]

but prettier is complaining about your formatting, can you pls fix the problems before we merge it?

[mauryapari]

Hey @vhoyer. fixed both of them. do check it.

[vhoyer]

the docs both seem to still have the final EOL problem

[mauryapari]

Hey. not sure what happened earlier. i have added it again for those.

[vhoyer]

 $ /home/runner/work/eslint-plugin-vuejs-accessibility/eslint-plugin-vuejs-accessibility/node_modules/.bin/prettier --check '**/*'
Checking formatting...
[warn] docs/rules/no-aria-hidden-on-focusable.md
[warn] docs/rules/no-role-presentation-on-focusable.md
[warn] src/rules/__tests__/no-aria-hidden-on-focusable.test.ts
[warn] src/rules/__tests__/no-role-presentation-on-focusable.test.ts
[warn] src/rules/no-aria-hidden-on-focusable.ts
[warn] src/rules/no-role-presentation-on-focusable.ts
[warn] src/utils/hasFocusableElement.ts
[warn] Code style issues found in 7 files. Run Prettier with --write to fix.```

[mauryapari]

okay i believe it should be resolved now. have a look.

[vhoyer]

it would be better if we removed the comments, no?

[mauryapari]

done

[vhoyer]

published https://github.com/vue-a11y/eslint-plugin-vuejs-accessibility/releases/tag/v2.4.0