vakata / jstree

jquery tree plugin
http://jstree.com
MIT License
5.15k stars 1.38k forks source link

[Accessibility] Additional fix for #2431. Remove unnecessary accessible label from li attribute #2465

Closed Karkhutvy closed 3 years ago

Karkhutvy commented 3 years ago

It's additional fix for https://github.com/vakata/jstree/issues/2431. As <li> element has role='none' it should be hidden from assistive technologies. But this element still has accessible label (aria-labelldby) attribute. Some assistive technologies might ignore role='none' if element has accesible name. In some cases this can cause unwanted behavior, for example:

To fix this aria-labelldby attribute cold be removed from <li> element to not confuse assistive technologies. <Li> element should be hideen from assistive technologies and no need any aria attrbutes. Our focusable element is <a> and accessible name is already exposed by this element.