Skip to content

Conversation

@DBowen33
Copy link
Contributor

@DBowen33 DBowen33 commented May 22, 2024

Fixed an issue to where leaf nodes were wrongly being assigned the aria-expanded attribute, which could confuse screen reader users.

https://screencast.googleplex.com/cast/NjMzNDkwNjM4MzcyODY0MHxkMTY5ZTdkNC05Mw

Fixes #21922
Fixes #21143

@DBowen33 DBowen33 marked this pull request as ready for review May 22, 2024 22:22
@DBowen33 DBowen33 requested a review from a team as a code owner June 7, 2024 19:15
@DBowen33 DBowen33 requested review from wagnermaciel and removed request for a team June 7, 2024 19:15
@DBowen33 DBowen33 closed this Jun 7, 2024
@DBowen33 DBowen33 reopened this Jun 8, 2024
@DBowen33 DBowen33 requested a review from andrewseguin June 8, 2024 01:32
@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 Jun 17, 2024
@andrewseguin andrewseguin merged commit 43b8dcb into angular:main Jun 17, 2024
andrewseguin pushed a commit that referenced this pull request Jun 17, 2024
…leaf node (#29096)

* fix(material/tree): fixed unit tests

fixed unit tests

Fixes #21922

* fix(material/tree): updated public api file

updated public api file

Fixes #21922

(cherry picked from commit 43b8dcb)
andrewseguin added a commit to andrewseguin/components that referenced this pull request Jun 17, 2024
andrewseguin added a commit that referenced this pull request Jun 17, 2024
andrewseguin added a commit that referenced this pull request Jun 17, 2024
… in the leaf node (#29096)" (#29272)

This reverts commit 43b8dcb.

(cherry picked from commit 004a943)
DBowen33 added a commit to DBowen33/components that referenced this pull request Jul 3, 2024
…leaf node (angular#29096)

* fix(material/tree): fixed unit tests

fixed unit tests

Fixes angular#21922

* fix(material/tree): updated public api file

updated public api file

Fixes angular#21922
DBowen33 added a commit to DBowen33/components that referenced this pull request Jul 12, 2024
…leaf node (angular#29096)

* fix(material/tree): fixed unit tests

fixed unit tests

Fixes angular#21922

* fix(material/tree): updated public api file

updated public api file

Fixes angular#21922
@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 Jul 18, 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.

bug(cdk-tree): aria-expanded attribute should not appear in the leaf node Tree does not follow the expected ARIA structure

2 participants