act-rules / act-rules.github.io

Accessibility conformance testing rules for HTML
https://act-rules.github.io/
Other
136 stars 68 forks source link

new rule: ARIA required ID references exist #2041

Closed WilcoFiers closed 1 year ago

WilcoFiers commented 1 year ago

This PR is related to https://github.com/act-rules/act-rules.github.io/pull/2012, where we took the requirement for ID references to exist in the same tree out. Separating these tests into their own rule allows us to be a little more precise, and as the rules format requires for this to be atomic.

Need for Call for Review: 1 week (because this isn't actually new)


How to Review And Approve

MelSumner commented 1 year ago

This is great! I like the separation.

WilcoFiers commented 1 year ago

@AndyAtTV Thanks for bringing that up! I completely forgot to check the spec about that. I've changed it to use shadow DOM through a script tag.

WilcoFiers commented 1 year ago

I dismissed all reviews. To rereviewers, I think this is an editorial change so it doesn't need another call for review. Just 3 approvals.

MelSumner commented 1 year ago

Maybe silly question about shadow DOM - should it pass since it still wouldn't be usable for the end user?

I do see the value of checking that it exists, but I wonder about this.

WilcoFiers commented 1 year ago

@MelSumner The shadow DOM example should fail, as it create an accessibility issue. I'm not sure I understood your comment though. Do you want to elaborate?