-
Notifications
You must be signed in to change notification settings - Fork 354
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
New issues with vnu-jar 21.10.12 #2082
Comments
Here are the unique rules:
|
@sideshowbarker I think the treegrid parsing might be an upstream issue. It looks like the parent table does have the correct role, but it might be tripping up on the intermidiate |
Upstream where? |
Sorry, I mean in the Validator project. I was pinging here before opening an issue there, if you think the error is correct rather an a parsing issue |
Aha — sorry, my fault for not having seen where this issue was raised 😄 I thought it was in the validator repo…
I’m not sure yet — haven’t made time yet to investigate |
It seems as though the error is correct? The treegrid example has Am I misunderstanding the issue? Digging a bit further:
If you remove the
the validator throws this error:
Looking at the ARIA in HTML I think this last example should pass? Additionally, in the datepicker example nu validator detects a missing "file:/home/runner/work/aria-practices/aria-practices/examples/dialog-modal/datepicker-dialog.html":147.65-148.78: error: Element “td” is missing one or more of the following attributes: “aria-checked”, “role”. I’m not sure how nu validator works or how the checks are performed/updated. But it seems like there are some clashes? |
@jongund will file the bug to validator. ARIA Principle is "Therefore, authors MUST NOT apply aria-expanded, aria-posinset, aria-setsize, and aria-level to a row that descends from a table or grid, and user agents SHOULD NOT expose any of these four properties to assistive technologies unless the row descends from a treegrid." In other words, aria-expanded, aria-posinset, aria-setsize, and aria-level to a row is allowed at Treegrid. |
From the Spec: |
Submitted an issue to the VNU validator: |
Just tried updating locally to see if it was a simple upgrade, and got the following new issues
The text was updated successfully, but these errors were encountered: