jstree icon indicating copy to clipboard operation
jstree copied to clipboard

Accessibility Improvement for v4

Open ouija opened this issue 3 years ago • 2 comments

Hi there,

Thanks for providing such a great plugin! Can I make a suggestion/improvement for version 4 that the use of <i> tags (for icons) be replaced to something more "accessibly-friendly", such as a <span> tag?

This is to help reduce errors thrown with such accessibility tools like SiteImprove, which throw errors when <i> tags are found, and also notes that "even though 'i' tags are commonly used for inserting icons, this is considered bad practice."

Thank you!

ouija avatar Oct 27 '20 19:10 ouija

I will keep that in mind - thank you. The bigger issue is accessibility altogether for v.4, as the main renderer I intend to use is a flat one - I still do not know how it will work in screen readers.

vakata avatar Oct 28 '20 03:10 vakata

To elaborate:

How do you feel about the current default renderer in regard to accessibility. It is a flat list of divs - an infinite scroller. My understanding of the tree spec is that it will not work with screen readers. Do you think something can be done, or should we simply include a hierarchial renderer (similar to v.3) and leave it to the developer to choose the renderer? The latter is simpler, but it promotes bad practices - most developers will probably go with the faster and lighter infinite scroller and break accessibility. The v.4 code is in an early stage (but works) in a branch in this repo.

vakata avatar Oct 28 '20 18:10 vakata