Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Put aria-deprecated-role (from aria-roles) into its own rule #4069

Closed
Tracked by #4066
WilcoFiers opened this issue Jun 28, 2023 · 1 comment · Fixed by #4074
Closed
Tracked by #4066

Put aria-deprecated-role (from aria-roles) into its own rule #4069

WilcoFiers opened this issue Jun 28, 2023 · 1 comment · Fixed by #4074
Assignees
Labels
feat New feature or enhancement pr A pr has been created for the issue rule metadata Issues in the rule metadata code (lib/rules) rules Issue or false result from an axe-core rule
Milestone

Comments

@WilcoFiers
Copy link
Contributor

Part of #4066

The aria-deprecated-role check should run in its own rule, instead of as part of aria-roles.

@WilcoFiers WilcoFiers added feat New feature or enhancement rules Issue or false result from an axe-core rule rule metadata Issues in the rule metadata code (lib/rules) labels Jun 28, 2023
@WilcoFiers WilcoFiers added this to the Axe-core 4.8 milestone Jun 28, 2023
@straker straker self-assigned this Jun 29, 2023
@straker straker added the pr A pr has been created for the issue label Jun 29, 2023
@padmavemulapati
Copy link

Validated with the latest develop branch code base, and seeing that
aria-deprecated-role rule is running as its own rule not with aria-roles

Image

with the old version (4.7.0) failures can see under aria-roles

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or enhancement pr A pr has been created for the issue rule metadata Issues in the rule metadata code (lib/rules) rules Issue or false result from an axe-core rule
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants