-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Closed
hrueger/AGLight
#112Labels
AccessibilityThis issue is related to accessibility (a11y)This issue is related to accessibility (a11y)GThis is is related to a Google internal issueThis is is related to a Google internal issueP2The issue is important to a large percentage of users, with a workaroundThe issue is important to a large percentage of users, with a workaround
Description
Bug, feature request, or proposal:
Bug
What is the expected behavior?
aria-required should only be used when we have valid role attribute.
What is the current behavior?
I am using mdc-chips backed mat-chip-grid implementation. mat-chip-grid component is adding aria-required=false without a role when grid has rows.
What are the steps to reproduce?
- Import MDC chips based mat-chip-grid implementation
- Paste following template
<mat-chip-grid #chipList>
<mat-chip-row [removable]="true">
chip
</mat-chip-row>
<input [matChipInputFor]="chipList"
[matChipInputAddOnBlur]="true"/>
</mat-chip-grid>
- Inspect browser DOM to see mat-chip-grid element has aria-required=false even though it doesn't have role attribute.
What is the use-case or motivation for changing an existing behavior?
This is a potential accessibility violation reported by axe. aria-required=false doesn't make sense for non form elements: https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/ARIA_Techniques/Using_the_aria-required_attribute
Metadata
Metadata
Assignees
Labels
AccessibilityThis issue is related to accessibility (a11y)This issue is related to accessibility (a11y)GThis is is related to a Google internal issueThis is is related to a Google internal issueP2The issue is important to a large percentage of users, with a workaroundThe issue is important to a large percentage of users, with a workaround