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

fix: Remove deprecate rules from accessibility scanner #2589

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

v-sharmachir
Copy link

Details

Below rules are deprecated by axe-core and are disabled in accessibility insights chrome extension. Removing these rules from accessibility scanner service as well.
aria-roledescription: Deprecated in axe-core 4.7.0. Refer False positives when using aria-roledescription · Issue #3649 · dequelabs/axe-core
duplicate-id-active: Deprecated in axe-core 4.8.0. Refer Update duplicate-id rules to align with WCAG 2.2 · Issue #3892 · dequelabs/axe-core

Motivation

2232180

Context

Pull request checklist

  • Addresses an existing issue: Fixes 2232180
  • Added relevant unit test for your changes. (yarn test)
  • Verified code coverage for the changes made. Check coverage report at: <rootDir>/test-results/unit/coverage
  • Ran precheckin (yarn precheckin)
  • Validated in an Azure resource group

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants