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

Issue 106: The <form> element should only map to FORM landmark with restriction of accessible name #107

Merged
merged 2 commits into from
Oct 10, 2017

Conversation

jongund
Copy link
Contributor

@jongund jongund commented Aug 29, 2017

Updated the form element mapping to only be a form landmark when it html as an accessible name

@jongund
Copy link
Contributor Author

jongund commented Sep 14, 2017

I am working on some landmark rules with Deque labs and having this requirement in the spec would similify rules relating to top level landmarks (e.g. main, banner and contentinfo)

updated link
@stevefaulkner stevefaulkner merged commit bd7cfed into w3c:gh-pages Oct 10, 2017
@stevefaulkner
Copy link
Contributor

thanks @jongund

@joanmarie
Copy link
Contributor

joanmarie commented Oct 10, 2017

What does "no corresponding role" mean with respect to platforms? Does it mean "pruned from the tree" or is it mapped to a platform accessibility role? I don't have strong feelings either way, as long as it doesn't show up as ATK_ROLE_UNKNOWN on my platform. 😄 But I do think more guidance is called for in order to ensure implementors know what to do for nameless HTML form elements.

Speaking just for my platform, if it is not pruned from the accessibility tree, possible mappings would be:

  • ATK_ROLE_SECTION (which is the mapping used for the div element and other generic block elements which need to remain in the accessibility tree but have no semantic meaning)
  • ATK_ROLE_FORM (which is not the same as the mapping for the ARIA form role, which should be exposed as an ATK_ROLE_LANDMARK).

Thoughts?

EDIT: Compare to what was done for the footer element which has "no corresponding role" in the ARIA row, AND explicit platform mappings beneath.

@stevefaulkner
Copy link
Contributor

@joanmarie I will add extra suggested mappings and we can take it from there, thanks!

@stevefaulkner
Copy link
Contributor

fixed via 2724b29

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 this pull request may close these issues.

3 participants