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(focus-order-semantics): Add ARIA role article to list of valid roles for scrollable regions #3927

Conversation

philipjia123
Copy link
Contributor

fix(check): Add ARIA role article to list of valid roles for scrollable regions in valid-scrollable-semantics check and update tests.

Closes: #3905

@philipjia123 philipjia123 changed the title Codespace philipjia123 laughing space pancake 7pr9w4q7473p7jj fix(check): Add ARIA role article to list of valid roles for scrollable regions Mar 1, 2023
@straker
Copy link
Contributor

straker commented Mar 6, 2023

@philipjia123 Thanks for the pr. Did you mean to open this as a draft?

@philipjia123 philipjia123 marked this pull request as ready for review March 7, 2023 18:39
@philipjia123 philipjia123 requested a review from a team as a code owner March 7, 2023 18:39
@philipjia123
Copy link
Contributor Author

@philipjia123 Thanks for the pr. Did you mean to open this as a draft?

Oh d'oh, sorry didn't realize it was just sitting as a draft!

Copy link
Contributor

@straker straker left a comment

Choose a reason for hiding this comment

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

Awesome! Good work.

Reviewed for security.

@straker straker changed the title fix(check): Add ARIA role article to list of valid roles for scrollable regions fix(focus-order-semantics): Add ARIA role article to list of valid roles for scrollable regions Mar 7, 2023
@straker straker merged commit f029271 into dequelabs:develop Mar 7, 2023
@philipjia123
Copy link
Contributor Author

Wonderful, thanks Steven!

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.

focus-order-semantics false positive on role="article" elements
2 participants