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.

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

AccName: tests/todos in accname/name/comp_labelledby #35

Closed
Tracked by #3
cookiecrook opened this issue Apr 27, 2023 · 4 comments · Fixed by web-platform-tests/wpt#42522
Closed
Tracked by #3

AccName: tests/todos in accname/name/comp_labelledby #35

cookiecrook opened this issue Apr 27, 2023 · 4 comments · Fixed by web-platform-tests/wpt#42522
Assignees

Comments

@cookiecrook
Copy link
Collaborator

cookiecrook commented Apr 27, 2023

AccName: Remaining tests/todos in accname/name/comp_labelledby.html

https://w3c.github.io/accname/#comp_labelledby

Some follow-on work is blocked by w3c/accname#209
so additional tests will come in #68

@cookiecrook cookiecrook changed the title AccName: Remaining tests/todos in accname/name/comp_labelledby.html AccName: tests/todos in accname/name/comp_labelledby Apr 27, 2023
@thinkbulecount2
Copy link

@cookiecrook Please assign to me. Thank you!

@cookiecrook
Copy link
Collaborator Author

Sent admin request to add you to the right lists.

@cookiecrook
Copy link
Collaborator Author

cookiecrook commented Oct 13, 2023

First question for the ARIA/AccName group... All three implementations fail this test, but they all fail it the same way, which usually makes me doubt my test expectation:

<nav aria-labelledby="r1" class="ex" data-expectedlabel="inception cubed" data-testname="nav with straight recursion">
  <h2 id="r1" aria-labelledby="r2">1st FAIL IF INCLUDED</h2>
  <span id="r2" aria-labelledby="r3">2nd FAIL IF INCLUDED</span>
</nav>
<span id="r3" aria-label="inception cubed">3rd FAIL IF INCLUDED</span>

Gecko, WebKit, Chromium all agree on the computedlabel here:

FAIL message: expected "inception cubed" but got "1st FAIL IF INCLUDED"

While I hope no one will write code like that in production, I believe I am reading the labelledby portion of the algorithm correctly, and that the recursion step allows chaining together references like this. Do the editors and ARIA implementors agree that the implementations are not to spec? Do implementors have concerns this recursion step of the algorithm?

@cookiecrook
Copy link
Collaborator Author

Confirmed there's an ambiguity and filed issue #209 against AccName... will comment or remove that test in the meantime.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants