-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add support for setting DOM focus via the wiki testable statements #13
Comments
Update format of "Create Session" command
Updated link for make_tests.pl: https://github.com/w3c/web-platform-tests/blob/master/wai-aria/tools/make_tests.pl#L282 |
Syntax in spec for event: https://spec-ops.github.io/atta-api/index.html#event |
Tests in the wiki using the syntax (sans description, but...), start here: https://www.w3.org/wiki/ARIA_1.1_Testable_Statements#application_activedescendant |
Sample output when using make_tests.pl to generate tests from wiki:
|
We need a wiki syntax for the event... which I thought we had specified. |
Isn't the syntax we specified what I commented above? |
No. That's the JSON structure. |
Question: do you think that the JSON you are putting in the activedescendant tests actually works? ATTAcomm has support for it - it calls focus. But my understanding was that focus was not portable and didn't necessarily emulate a user setting focus reliably. |
An event record in the wiki will generate corresponding JSON in the test now. References #13
It doesn't cause an accessible focus event to be emitted, but if I remove that step from the json, I get new failures, namely that the element lacks STATE_FOCUSED. |
An event record in the wiki will generate corresponding JSON in the test now. References #13
We have several tests (e.g. aria-activedescendant) which require setting DOM focus on an element. And I can add this manually to an already generated test as a step, e.g.:
Is this the desired way to do DOM focus? If not, what is? But if so.... Looking at make_tests.pl:
Since we are currently doing wiki-based testable statements, it would be great to solve this problem so the aria-activedescendant tests don't have to be run manually.
The text was updated successfully, but these errors were encountered: