react-virtualized icon indicating copy to clipboard operation
react-virtualized copied to clipboard

Semantically correct HTML

Open crshmk opened this issue 5 years ago • 3 comments

We like this tool very much, but we ran into a wall when the accessibility team reviewed our app. That the ReactVirtualized__Grid__innerScrollContainer in Grid.js is a div, rather than a ul, that contains lis makes it confusing for users on a screen reader. Moreover, a list of lis inside a div it is not semantically correct html. Could we please change this div

<div
  className="ReactVirtualized__Grid__innerScrollContainer"

in Grid.js to a ul?

This does presume list items would be a list of lis. If developers need the ability to render divs or some other element in the renderer, could we make the element type configurable? At the moment we see no way to achieve a semantically correct list, i.e. a ul with lis as direct children.

crshmk avatar Apr 01 '20 21:04 crshmk

Bump for improved accessibility

tavgordon avatar Sep 30 '21 15:09 tavgordon

Would also like to bump this.

jbean96 avatar Mar 28 '22 19:03 jbean96

Any solution for this?

RubenZx avatar May 02 '22 08:05 RubenZx