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

Editorial clarification suggested for AXAPI mappings for audio and video elements #80

Closed
joanmarie opened this issue Jan 23, 2017 · 2 comments
Assignees

Comments

@joanmarie
Copy link
Contributor

Under audio and video for AXAPI, it says:

Note: UI controls within the audio element exposed as AXToolbar.

I assume that means the controls (e.g. play and pause buttons) are exposed as descendants of an accessible object with a role of AXToolbar. Or to put it another way, they are exposed on a toolbar. But reading the statement quoted above, it sounds like the play and pause button themselves have role of AXToolbar. Can this be clarified as an editorial change?

jasonkiss added a commit that referenced this issue Feb 8, 2017
@jasonkiss jasonkiss self-assigned this Feb 8, 2017
@jasonkiss
Copy link
Contributor

Good call. Thanks @joanmarie. I've tried to clarify how the sub-DOM controls are to be exposed in the above commit ab5855f.

How does the new language for the audio and video mappings work for you? I basically borrowed and updated the language from the UIA mappings that were trying to express the same thing. So @boggydigital might want to take a peek, as well.

I also took the opportunity to update the AXSubrole and AXRoleDescription values based on what's implemented in webkit.

Should we be making similar overtures about descendant UI controls for IA2 and ATK? @asurkov?

Note that @cyns asked the more general question just over a year ago, in issue #3, about how we deal with elements that have sub-trees of controls or other other objects. Further discussion on this question should happen over there on issue #3.

@jasonkiss
Copy link
Contributor

Have merged changes. Can revisit for other AAPIs if needed.

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

No branches or pull requests

2 participants