search

dequelabs / axe-core

Accessibility engine for automated Web UI testing
https://www.deque.com/axe/
Mozilla Public License 2.0
5.76k stars 747 forks source link

isVisibleToScreenReaders should consider aria-owns when looking at aria-hidden #3689

Open WilcoFiers opened 1 year ago

WilcoFiers commented 1 year ago

Axe-core's isVisibleToScreenReaders method currently considers the li in the following code snippet as hidden. This doesn't seem to be how most (all?) browsers build their accessibility tree though. aria-owns seems to take priority over aria-hidden.

<ul aria-owns="li"></ul>
<div aria-hidden="true">
  <li id="li">Hello world</li>
</div>
WilcoFiers commented 1 year ago

Here's the corresponding issue I filed with WAI-ARIA on the subject: https://github.com/w3c/aria/issues/1818