Layer's related aria-haspopup misplaced/misused

[Malvoz]

Currently, aria-haspopup is improperly set on the layer list's and control's container div element:

Leaflet/src/control/Control.Layers.js

Line 180 in 983092a

container.setAttribute('aria-haspopup', true);

The aria-haspopup description suggests that the attribute should be set on the element that triggers additional content (which is <a class="leaflet-control-layers-toggle"> in this case).

Indicates the availability and type of interactive popup element, such as menu or dialog, that can be triggered by an element.

It might be a good idea to check out the different patterns in the W3C ARIA Practices documentation to establish what sort of pattern would best fit the layer control (for example the disclosure widget).

[johnd0e]

Currently, aria-haspopup is improperly set on the layer list's and control's container div element:

See d18f624.
It's sole function is proper functioning on Windows phones, and it seems ok.

[Malvoz]

@johnd0e Interesting, I found the explainer from Microsoft (and related mailing list conversations W3C, WebAIM).

The issue withstanding is that the triggering element does not have aria-haspopup, which per definition "indicates the availability" of popup content. When a user with a screen reader focuses the layer control, the screen reader wont convey information from ARIA attributes in (arbitrary) parent elements.

[Malvoz]

Even if aria-haspopup was properly conveyed to users of assistive technology by moving it to the <a> control, it would be incorrect to do so because aria-haspopup has requirements on the type of content it controls:

aria-haspopup:

Authors MUST ensure that the role of the element that serves as the container for the popup content is menu, listbox, tree, grid, or dialog, and that the value of aria-haspopup matches the role of the popup container.

(note that the value true as currently used is treated as menu)

None of which I believe are appropriate for the layer controls list.

I'm closing this, superseded by #7526.