Skip to content

Commit

Permalink
Deprecate aria-roledescription
Browse files Browse the repository at this point in the history
  • Loading branch information
codeofdusk committed Jul 31, 2023
1 parent 101f15b commit ebcff53
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ export class LoadAssessmentDataSchemaProvider {
private setDeprecatedRequirementProperties(schema: any) {
const deprecatedRequirements = [
{ assessmentKey: 'automated-checks', requirementKey: 'duplicate-id' },
{ assessmentKey: 'automated-checks', requirementKey: 'scrollable-region-focusable' },
{ assessmentKey: 'automated-checks', requirementKey: 'aria-roledescription' },
];
deprecatedRequirements.forEach(requirement => {
if (this.getAssessments(schema)[requirement.assessmentKey] === undefined) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -66,8 +66,8 @@ describe(LoadAssessmentDataSchemaProvider, () => {
properties:
assessment.key === 'automated-checks'
? {
'aria-roledescription': stepProperties,
'duplicate-id': stepProperties,
'scrollable-region-focusable': stepProperties,
}
: {},
type: ['object', 'null'],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,8 @@ exports[`getRuleInclusions getRuleInclusions matches snapshotted list of product
"status": "included",
},
"aria-roledescription": {
"status": "included",
"reason": "disabled in axe config",
"status": "excluded",
},
"aria-roles": {
"status": "included",
Expand Down

0 comments on commit ebcff53

Please sign in to comment.