Skip to content

Conversation

zarend
Copy link
Contributor

@zarend zarend commented Mar 7, 2023

Allow keyboard navigation to disabled options of Select and Autocomplete
component. Align with WAI ARIA instructions for list options from
Developing a Keyboard
Interface
.

Fix accessibility issues where some screen readers are not able to navigate
to disabled options.

The user can focus disabled options using keyboard, but the user cannot
click disabled options.

@zarend zarend force-pushed the disabled-mat-option branch from 88a9c8b to 96c3d59 Compare March 10, 2023 20:59
@zarend zarend changed the title fix(material/select, material/autocomplete): keyboard navigates to di… fix(material/core): allow keyboard navigation to disabled options Mar 10, 2023
@zarend zarend marked this pull request as ready for review March 10, 2023 21:07
@zarend zarend added the target: minor This PR is targeted for the next minor release label Mar 10, 2023
@devversion devversion removed their request for review March 13, 2023 14:01
@zarend zarend force-pushed the disabled-mat-option branch from 96c3d59 to 552f7a5 Compare March 14, 2023 18:48
@zarend zarend added the action: merge The PR is ready for merge by the caretaker label Mar 14, 2023
@zarend zarend force-pushed the disabled-mat-option branch 2 times, most recently from a8050ab to 99f7607 Compare March 22, 2023 21:23
@zarend
Copy link
Contributor Author

zarend commented Mar 22, 2023

since this was reviewed, refactored option.scss so that selector targets only what it needs to target rather than using * selector

@zarend zarend force-pushed the disabled-mat-option branch from 99f7607 to 11052ed Compare March 23, 2023 20:37
Allow keyboard navigation to disabled options of Select and Autocomplete
component. Align with WAI ARIA instructions for list options from
[Developing a Keyboard
Interface](https://www.w3.org/WAI/ARIA/apg/practices/keyboard-interface/).

Fix accessibility issues where some screen readers are not able to navigate
to disabled options.

The user can focus disabled options using keyboard, but the user cannot
click disabled options.
@zarend zarend force-pushed the disabled-mat-option branch from 11052ed to 97dd43d Compare March 23, 2023 20:37
@zarend
Copy link
Contributor Author

zarend commented Mar 23, 2023

Fix selector for disabled pseudo checkbox. 🤦 ✔️

@angular-robot angular-robot bot merged commit 6ad47bf into angular:main Mar 25, 2023
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Apr 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Accessibility This issue is related to accessibility (a11y) action: merge The PR is ready for merge by the caretaker area: material/autocomplete area: material/core area: material/select target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants