Skip to content

fix: port CL that fixes ARIA tree impl for macOS#22367

Merged
jkleinsc merged 2 commits into
masterfrom
fix-aria-trees-master
Feb 26, 2020
Merged

fix: port CL that fixes ARIA tree impl for macOS#22367
jkleinsc merged 2 commits into
masterfrom
fix-aria-trees-master

Conversation

@loc

@loc loc commented Feb 25, 2020

Copy link
Copy Markdown
Contributor

Description of Change

Port a CL from Chrome that fixes ARIA trees for macOS VoiceOver. There's more detail in the CL and in the linked bug, but ARIA trees were manifesting as empty tables, which is not exactly useful to screenreader users.

https://chromium-review.googlesource.com/c/chromium/src/+/2062913

Checklist

Release Notes

Notes: Fixed ARIA role="tree" for macOS VoiceOver

@electron-cation electron-cation Bot added new-pr 🌱 PR opened recently and removed new-pr 🌱 PR opened recently labels Feb 25, 2020
@loc loc marked this pull request as ready for review February 26, 2020 19:15
@loc loc requested a review from a team as a code owner February 26, 2020 19:15
@loc

loc commented Feb 26, 2020

Copy link
Copy Markdown
Contributor Author

Chrome merged it, now ready for review!

@jkleinsc jkleinsc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice!

@jkleinsc jkleinsc merged commit fc54db1 into master Feb 26, 2020
@release-clerk

release-clerk Bot commented Feb 26, 2020

Copy link
Copy Markdown

Release Notes Persisted

Fixed ARIA role="tree" for macOS VoiceOver

@jkleinsc jkleinsc deleted the fix-aria-trees-master branch February 26, 2020 22:34
@trop

trop Bot commented Feb 26, 2020

Copy link
Copy Markdown
Contributor

I was unable to backport this PR to "9-x-y" cleanly;
you will need to perform this backport manually.

@trop

trop Bot commented Feb 26, 2020

Copy link
Copy Markdown
Contributor

I was unable to backport this PR to "7-1-x" cleanly;
you will need to perform this backport manually.

@trop trop Bot removed the target/7-1-x label Feb 26, 2020
@trop

trop Bot commented Feb 26, 2020

Copy link
Copy Markdown
Contributor

I was unable to backport this PR to "8-x-y" cleanly;
you will need to perform this backport manually.

@trop

trop Bot commented Feb 27, 2020

Copy link
Copy Markdown
Contributor

@loc has manually backported this PR to "9-x-y", please check out #22421

@trop

trop Bot commented Feb 27, 2020

Copy link
Copy Markdown
Contributor

@loc has manually backported this PR to "8-x-y", please check out #22423

@trop

trop Bot commented Feb 27, 2020

Copy link
Copy Markdown
Contributor

@loc has manually backported this PR to "7-1-x", please check out #22424

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants