accessibility-insights-web icon indicating copy to clipboard operation
accessibility-insights-web copied to clipboard

False positive? Ensure elements that have scrollable content are accessible by keyboard

Open jzazo opened this issue 1 year ago • 2 comments
trafficstars

Describe the bug

I am running the accessibility report extension and I get flagged an error: scrollable-region-focusable: Elements with scrollable content must be accessible by keyboard. It affects the search box, enabled via plugin.

Path: .md-search__scrollwrap

Snippet:
- <div class="md-search__scrollwrap"> <div class="md-search-result" data-md-component="search-result">
- <div class="md-search-result__meta">Type to start searching</div> <ol class="md-search-result__list" role="presentation"></ol> </div> 

Fix ONE of the following:
- Element should have focusable content
- Element should be focusable

Adding a tabindex="0" to the div section <div class="md-search__scrollwrap"> removes the error.

However, I created this issue in the material-mkdocs project and you can navigate search results with the keyboard, and the author claims that the flag is a false positive.

To Reproduce Steps to reproduce the behavior:

  1. Go to https://microsoft.github.io/rats/
  2. Run the accessibility-insights-web extension. The error relates to the search box. If you search commands you can scroll through results, so the error is probably a false positive.

Expected behavior

If this is a false positive, an error should not be flagged.

Screenshots

Screenshot 2024-05-18 145934 Screenshot 2024-05-18 145955

Context (please complete the following information)

  • OS Name & Version: Windows 11
  • Browser Version: Microsoft Edge Version 125.0.2535.51 (Official build) (64-bit)
  • Target Page: https://microsoft.github.io/rats/

Are you willing to submit a PR?

No.

Did you search for similar existing issues?

Yes.

jzazo avatar May 18 '24 14:05 jzazo

This issue requires additional investigation by the Accessibility Insights team. When the issue is ready to be triaged again, we will update the issue with the investigation result and add "status: ready for triage". Thank you for contributing to Accessibility Insights!

@jzazo We have opened https://github.com/dequelabs/axe-core/issues/4470 for axe-core. We will update in the extension once we have fix from upstream. Thanks

v-viyada avatar May 21 '24 20:05 v-viyada

This issue requires an external fix that is tracked here: https://github.com/dequelabs/axe-core/issues/4470. Closing this issue as this will not be fixed directly in Accessibility Insights for Web. Once fixed in axe-core, the team can integrate the updated axe-core.

joestegman425 avatar Jan 28 '25 19:01 joestegman425