d3/d3-brush

Swap "e" and "w" resize controls to align DOM with actual view

zakimaksyutov opened this issue · 1 comments

zakimaksyutov commented

Right now "east resize" goes first (example from v3 but v4 has the same order):

image

Impact

When extending resize controls to be focusable, focus/screen reader still follows DOM order and as a result goes in reverse order:

brush-taborder2

Potential fix

Swap "e" and "w" in array of handles to align DOM order with actual view.

mbostock commented

Thanks for the suggestion and sorry for the delay! This has been fixed.

  • 👍1