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

Features should have accessible names for discernability #316

Closed
Malvoz opened this issue Feb 24, 2021 · 7 comments
Closed

Features should have accessible names for discernability #316

Malvoz opened this issue Feb 24, 2021 · 7 comments

Comments

@Malvoz
Copy link
Member

Malvoz commented Feb 24, 2021

The features are indistinguishable to someone relying on a screen reader as they don't have an accessible name. <path tabindex="0"> is announced as graphic in Chrome (blank in Firefox):

Keyboard.Interaction.Test.-.Google.Chrome.2021-02-01.01-37-.mp4

Originally posted by @Malvoz in #270 (comment)

@Malvoz
Copy link
Member Author

Malvoz commented Feb 24, 2021

I see that the <path> element supports a child <title> element that provides an accessible name. If we can come up with a convention for providing a similar accessible name for <feature>, we could generate the <path><title> from it.

Originally posted by @prushforth in #270 (comment)

@Malvoz
Copy link
Member Author

Malvoz commented Feb 24, 2021

Given the extremely generic properties model that <feature> has, I wonder if we should add a <title> element child of <properties> as a special case for accessible names purposes similar to svg. In the polyfill we can map that to <title> or what have you. I'll open an issue on this latter discussion in the MapML and UCR document repos.

Originally posted by @prushforth in #270 (comment)

@Malvoz Malvoz changed the title Provide a way for authors to label features Features should have accessible names for discernability Feb 24, 2021
@prushforth
Copy link
Member

I believe that title is a global HTML attribute, so maybe it makes sense to respect / use a <feature title> as an accessible name. We don't have to worry about conflicts between a <title> and <feature title>.

@Malvoz
Copy link
Member Author

Malvoz commented Feb 25, 2021

In that case we should consider label as opposed to title.

@Malvoz
Copy link
Member Author

Malvoz commented Feb 25, 2021

label is element-specific per element's definitions, and more encouraging for authors to use as opposed to title which feels more "optional" since it's a global. title also implies the native tooltip on hover.

@prushforth
Copy link
Member

A little googling on the title attribute reveals it is considered an accessibility problem. So scratch that proposal.

prushforth added a commit to prushforth/MapML-Specification that referenced this issue Mar 1, 2021
prushforth added a commit to prushforth/MapML-Specification that referenced this issue Mar 1, 2021
prushforth added a commit to prushforth/MapML-Specification that referenced this issue Mar 1, 2021
…L.js#316.

Change mapml vocabulary to the xhtml namespace.

Fix error in URL introduced in previous commit.
prushforth added a commit to Maps4HTML/MapML-Specification that referenced this issue Mar 2, 2021
* Get rid of references to MicroXML, update Abstract

* Add link to UCR

* Add text-valued <title> element child of <feature> per Maps4HTML/MapML.js#316.

Change mapml vocabulary to the xhtml namespace.

Fix error in URL introduced in previous commit.

* Rename <feature><title> to <feature><featurecaption>

* Add spec description of <featurecaption>. Remove obsolete sch file.

* Revise feature, featurecaption, properties specifications, links.

* Delete redundant paragraph. Fix incorrect link reference. Remove redundant 
text description.
@prushforth
Copy link
Member

This is resolved by opting for a <feature><featurecaption> element for the moment.

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

No branches or pull requests

2 participants