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

3837 - aria-hidden adjustments #570

Merged
merged 14 commits into from
Jan 10, 2022
Merged

3837 - aria-hidden adjustments #570

merged 14 commits into from
Jan 10, 2022

Conversation

shunguoy
Copy link
Contributor

@shunguoy shunguoy commented Dec 9, 2021

aria-hidden initial change

Copy link
Member

@tombrunet tombrunet left a comment

Choose a reason for hiding this comment

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

Noted various comments. Please review.

@shunguoy shunguoy marked this pull request as ready for review December 16, 2021 19:37
@tombrunet tombrunet self-assigned this Dec 16, 2021
@tombrunet
Copy link
Member

@shunguoy I changed the function name to isNodeHiddenFromAT since isNodeHidden was misleading to me given that the node could be visible.

tombrunet
tombrunet previously approved these changes Dec 17, 2021
@tombrunet tombrunet changed the title Syan 3837 3837 - aria-hidden adjustments Dec 17, 2021
@tombrunet
Copy link
Member

@shunguoy See my note on Slack - seems you squashed a change, but Git isn't showing the deletion - not sure why.

@marcjohlic marcjohlic added engine Issues in the accessibility-checker-engine component Ready for QA labels Jan 10, 2022
@ErickRenteria ErickRenteria merged commit 6a887c3 into master Jan 10, 2022
@shunguoy shunguoy deleted the syan-3837 branch January 14, 2022 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engine Issues in the accessibility-checker-engine component Ready for QA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants