Skip to content

Replace ARIA reflection attribute ariaErrorMessageElement with ariaErrorMessageElements - #37856

Merged
rniwa merged 1 commit into
web-platform-tests:masterfrom
rniwa:replace-ariaErrorMessageElement-with-ariaErrorMessageElements
Jan 12, 2023
Merged

rniwa merged 1 commit into
web-platform-tests:masterfrom
rniwa:replace-ariaErrorMessageElement-with-ariaErrorMessageElements

Conversation

@rniwa

@rniwa rniwa commented Jan 11, 2023

Copy link
Copy Markdown
Contributor

This is WPT updates for w3c/aria#1802

@annevk annevk 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.

I think it would be good to add a historical test that asserts ariaErrorMessageElement is not implemented.

@rniwa

rniwa commented Jan 11, 2023

Copy link
Copy Markdown
Contributor Author

I think it would be good to add a historical test that asserts ariaErrorMessageElement is not implemented.

Good point. Adding that.

@rniwa
rniwa force-pushed the replace-ariaErrorMessageElement-with-ariaErrorMessageElements branch from 2c550ee to 2948659 Compare January 11, 2023 08:46

@annevk annevk 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.

Thanks!

@rniwa
rniwa force-pushed the replace-ariaErrorMessageElement-with-ariaErrorMessageElements branch from 2948659 to ce6f026 Compare January 11, 2023 08:52

@mrego mrego 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.

LGTM, thanks for updating the tests.

@rniwa
rniwa merged commit b2408e7 into web-platform-tests:master Jan 12, 2023
@rniwa
rniwa deleted the replace-ariaErrorMessageElement-with-ariaErrorMessageElements branch January 12, 2023 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants