Skip to content

Replace ARIA reflection attribute ariaErrorMessageElement with ariaErrorMessageElements#37856

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

Replace ARIA reflection attribute ariaErrorMessageElement with ariaErrorMessageElements#37856
rniwa merged 1 commit intoweb-platform-tests:masterfrom
rniwa:replace-ariaErrorMessageElement-with-ariaErrorMessageElements

Conversation

@rniwa
Copy link
Copy Markdown
Contributor

@rniwa rniwa commented Jan 11, 2023

This is WPT updates for w3c/aria#1802

Copy link
Copy Markdown
Member

@annevk annevk left a comment

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
Copy link
Copy Markdown
Contributor Author

rniwa commented Jan 11, 2023

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
Copy link
Copy Markdown
Member

@annevk annevk left a comment

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
Copy link
Copy Markdown
Member

@mrego mrego left a comment

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