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.

Sign up for GitHub

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

Jump to bottom

Update to warn against potential redundant role for S element #1518

Closed
scottaohara opened this issue Jun 29, 2023 · 1 comment
Closed

Update to warn against potential redundant role for S element #1518

scottaohara opened this issue Jun 29, 2023 · 1 comment
Labels
engine Issues in the accessibility-checker-engine component priority-2 (med) Ready for QA Spec change When specs/versions are updated or change user-reported Issues identified outside of the core team

Comments

@scottaohara
Copy link

HTML AAM has been updated to map the s element to role=deletion with Chromium and Gecko having implemented this change, and an issue against webkit has been filed for them to do so as well.

ARIA in HTML is being updated w3c/html-aria#466 to reflect this change

Previously any role was allowed for the s element. That is still true but for the fact that like other elements with implicit roles it is now 'not recommended' for devs to use role=deletion on the element since it will already expose that.

The checker can surface this as a warning, as it does for other similar elements.
@scottaohara scottaohara mentioned this issue Jun 29, 2023
Merged
4 tasks
@shunguoy shunguoy added engine Issues in the accessibility-checker-engine component Spec change When specs/versions are updated or change priority-2 (med) labels Jul 10, 2023
@MHoov MHoov added the user-reported Issues identified outside of the core team label Jul 17, 2023
shunguoy added a commit that referenced this issue Jan 16, 2024
@shunguoy
update aria definition #1518
9425223
shunguoy added a commit that referenced this issue Jan 16, 2024
@shunguoy
update test cases for the S element #1518
5b5e74f
@shunguoy shunguoy mentioned this issue Jan 17, 2024
Merged
1 task
shunguoy added a commit that referenced this issue Jan 17, 2024
@shunguoy
update aria definition for aside role and test case #1518
574d852
@ErickRenteria
Copy link
Contributor

Validated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
engine Issues in the accessibility-checker-engine component priority-2 (med) Ready for QA Spec change When specs/versions are updated or change user-reported Issues identified outside of the core team
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@scottaohara @shunguoy @MHoov @ErickRenteria