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(material/select): remove incompatible aria-autocomplete attribute #29645

Conversation

essjay05
Copy link
Contributor

Fixes a bug reported in MatSelect where the usage of aria-autocomplete is not a valid attribute that can be used with aria role=listbox. Removes the aria-autocomplete attribute to fix the violation: aria-allowed-attr.

W3 Documentation on allowed attributes for role="listbox"

Fixes b/352496530

@essjay05 essjay05 requested a review from a team as a code owner August 26, 2024 21:35
@essjay05 essjay05 requested review from mmalerba and andrewseguin and removed request for a team August 26, 2024 21:35
@essjay05 essjay05 force-pushed the mat-select-remove-incompatible-aria-autocomplete-attr branch 4 times, most recently from 92e2d25 to a241fdb Compare September 5, 2024 17:02
@essjay05 essjay05 force-pushed the mat-select-remove-incompatible-aria-autocomplete-attr branch 2 times, most recently from 44afd16 to 501be8a Compare September 12, 2024 22:32
Fixes a bug reported in MatSelect where the usage of aria-autocomplete
is not a valid attribute that can be used with aria role=listbox.
Removes the aria-autocomplete attribute to fix the violation:
aria-allowed-attr.

Fixes b/352496530
@essjay05 essjay05 force-pushed the mat-select-remove-incompatible-aria-autocomplete-attr branch from 501be8a to bd1832e Compare September 18, 2024 17:39
@andrewseguin andrewseguin added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Sep 18, 2024
@andrewseguin andrewseguin removed the request for review from mmalerba September 18, 2024 17:50
@andrewseguin andrewseguin merged commit 6077663 into angular:main Sep 18, 2024
18 of 26 checks passed
andrewseguin pushed a commit that referenced this pull request Sep 18, 2024
…#29645)

Fixes a bug reported in MatSelect where the usage of aria-autocomplete
is not a valid attribute that can be used with aria role=listbox.
Removes the aria-autocomplete attribute to fix the violation:
aria-allowed-attr.

Fixes b/352496530

(cherry picked from commit 6077663)
@essjay05 essjay05 deleted the mat-select-remove-incompatible-aria-autocomplete-attr branch September 18, 2024 18:28
crisbeto added a commit to crisbeto/material2 that referenced this pull request Sep 19, 2024
After angular#29645, the assertion for `aria-autocomplete` isn't valid anymore.
crisbeto added a commit that referenced this pull request Sep 19, 2024
After #29645, the assertion for `aria-autocomplete` isn't valid anymore.
crisbeto added a commit that referenced this pull request Sep 19, 2024
After #29645, the assertion for `aria-autocomplete` isn't valid anymore.

(cherry picked from commit 886fb6d)
@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 Oct 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants