-
Notifications
You must be signed in to change notification settings - Fork 57
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
Review Accessibility Object Model #134
Comments
I would hold a little bit on this. We are still working through quite a few issues and I feel it's still premature to do a TAG review. |
Picked up at F2F Tokyo, agreed to defer until next F2F unless you want to bump it. |
@marcoscaceres Are we still holding on this? |
Yes. But expect a big announcement in the next few days. |
Suggesting we close and re-open when there's an update. Thoughts, @marcoscaceres? |
Also, is there a URL to the current spec? Original URL is 404. |
@dbaron the Phase 1 spec, and the repo |
Deferring to @alice, as Alice will know best when the spec will be ready for review. |
@marcoscaceres @alice are you ready for this review yet? |
Phase 1 is definitely ready for a look, and is available to experiment with in Chrome Beta per these instructions: https://groups.google.com/a/chromium.org/forum/#!topic/chromium-accessibility/UCfWokyvoj0 Your thoughts on the later phases (Phase 2 has a WIP draft; later phases are sketched out in concept only in the explainer) would also be welcome. |
Super. We'll take a look at Phase 2 in one of our upcoming teleconferences. Thanks for getting back to us! |
Taken up at Nice f2f. |
OK, coming back to this review! TPAC was a great chance to sync up, so thanks for your time there! Some point-by-point follow-ups on your last comments:
I participated in the discussion. Outcomes to surface as
Yep, that was a spirited discussion. I'm hopeful that the
I had this question answered when we chatted in-person. My understanding: if there will be a virtual a11y node, it must act as proxy for all the relevant state that would otherwise be on a regular Element. "Legacy" was just my way of grouping all pre-existing CSS-OM APIs. I hope we can help with the use-cases, but wondering if many of them can be addressed by the so-called "searchable invisible DOM" (which needs a better name BTW). Seems as if the virtual a11y node and related standarization of the a11y tree is still a way off, so not going to worry too much about it right now.
Ah, this is not what I expected. I thought the role IDL attribute might work more like input's |
I'm not sure I follow. The Computing the role can be complex; for example the ARIA 1.1 spec states:
In terms of a code example, this means: <table role="presentation">
<tr>
<td>All</td>
<td>these</td>
</tr>
<tr>
<td>elements</td>
<td>are</td>
</tr>
<tr>
<td colspan="2">presentational</td>
</tr>
</table> Because a table (grid) must own rows, and rows must own cells, the |
At Paris F2F: Discussed this further (the computed accessibility node concept). There is a strong parallel to the problems we've seen in Style computation and read-back problems that have long been a sore spot for performance. In this respect, my idea to read the banana.getA11yNode().then((n) => { console.log(n.role); }); |
Thanks for the extra feedback! I appreciate the suggestion about localising the |
The getComputedStyle() design is very strange. My best guess for why it exists that way was that separate working groups produced the CSS APIs from the Node APIs, and they didn't want to collaborate and/or didn't have I don't think there's anything special about promise return types that makes using them a cause for caution. So I tend to agree that putting it as a method on |
I suspect |
Thanks for the explanations! We'll definitely take that advice on board - keeping it promise based and moving it to Node. |
Hey Alice, Thanks so much for sticking with us through the [long] review! We appreciate all the time you've spent (and continue to sped) revising the various AOM docs and adjusting and reacting to everyone's feedback. We are looking forward to continuing the discussion. Regarding this particular issue, we think we've reached the end of our general feedback--that the design is trending in a good direction, and that it might be more productive to work on a review for a specific aspect of AOM once the spec reaches a good milestone for review. So, what we'd like to ask of you is to file a new TAG design-review issue for each "level" of AOM that is ready for more specific review. We look forward to continued involvement in the AOM project. Thanks so much and thanks for "flying TAG"! |
See: https://a11y-api.github.io/a11y-api/spec/
The text was updated successfully, but these errors were encountered: