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(angular/processflow): remove obsolete aria-expanded attribute #1623

Merged
merged 1 commit into from
Jul 27, 2022

Conversation

mhaertwig
Copy link
Collaborator

@mhaertwig mhaertwig commented Jul 26, 2022

The aria-expanded attribute is not valid for the tabpanel role (see e.g. here). It can be simply removed because selecting the tab with aria-selected in combination with an aria-controls pointing to the tabpanel is already enough.

Closes #1617

The `aria-expanded` attribute is not valid for the `tabpanel` role (see e.g. [here](https://www.w3.org/TR/wai-aria-1.2/#tabpanel)). It can be simply removed because selecting the tab with `aria-selected` in combination with an `aria-controls` pointing to the `tabpanel` is already enough.
@github-actions
Copy link

Copy link
Collaborator

@kyubisation kyubisation left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍
Should we maybe also open a PR at angular/components?

Copy link
Collaborator

@jeripeierSBB jeripeierSBB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mhaertwig mhaertwig merged commit c6aad0b into main Jul 27, 2022
@mhaertwig mhaertwig deleted the processflow-a11y branch July 27, 2022 13:06
sbb-design-systems-bot bot pushed a commit that referenced this pull request Jul 27, 2022
)

The `aria-expanded` attribute is not valid for the `tabpanel` role (see e.g. [here](https://www.w3.org/TR/wai-aria-1.2/#tabpanel)). It can be simply removed because selecting the tab with `aria-selected` in combination with an `aria-controls` pointing to the `tabpanel` is already enough.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Processflow not WCAG21 compatible because of misused aria-expanded attribute
3 participants