Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

scrollable-region-focusable: false positive for disabled multiline select / listbox #3692

Closed
1 task done
MichaelKetting opened this issue Oct 2, 2022 · 1 comment · Fixed by #3862
Closed
1 task done

scrollable-region-focusable: false positive for disabled multiline select / listbox #3692

MichaelKetting opened this issue Oct 2, 2022 · 1 comment · Fixed by #3862
Assignees
@WilcoFiers
Labels
fix Bug fixes pr A pr has been created for the issue rules Issue or false result from an axe-core rule
Milestone
Axe-core 4.7

Comments

@MichaelKetting
Copy link

Product

axe-core

Product Version

3.3.4 ... 4.4.2

Latest Version

  • I have tested the issue with the latest version of the product

Issue Description

Expectation

scrollable-region-focusable should not report for multiline select elements / listbox (when they are diabled)

Actual

I have a disabled select element showing a scrollbar because it contains more items than than it has rows configured and I get scrollable-region-focusable in my aXe result.

How to Reproduce

<!DOCTYPE html>
<html xmlns="http://www.w3.org/1999/xhtml" lang="en">
<head>
<title>Example</title>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8"/>
</head>
<body>
  <select size="2" name="select" disabled="disabled" aria-label="Test">
    <option value="First">First</option>
    <option value="Second">Second</option>
    <option value="Third">Third</option>
  </select>
</body>
</html>

Additional context

Since the browser does not allow focusing the element, it is not within the web-developer's scope to solve this issue for disabled elements.

see also #3691
@MichaelKetting MichaelKetting added the ungroomed Ticket needs a maintainer to prioritize and label label Oct 2, 2022
@MichaelKetting MichaelKetting mentioned this issue Oct 2, 2022
Closed
1 task
MichaelKetting added a commit to re-motion/Framework that referenced this issue Oct 2, 2022
@MichaelKetting
RM-7223 Add ignore rule for dequelabs/axe-core#3692
06fb759
MichaelKetting added a commit to re-motion/Framework that referenced this issue Oct 2, 2022
@MichaelKetting
RM-7223 Add ignore rule for dequelabs/axe-core#3692
3773170
MichaelKetting added a commit to re-motion/Framework that referenced this issue Oct 3, 2022
@MichaelKetting
RM-7223 Add ignore rule for dequelabs/axe-core#3692
0a63a98
@straker straker added fix Bug fixes rules Issue or false result from an axe-core rule and removed ungroomed Ticket needs a maintainer to prioritize and label labels Oct 3, 2022
@straker straker added this to the Axe-core 4.6 milestone Oct 3, 2022
@Zidious Zidious mentioned this issue Oct 6, 2022
Closed
@WilcoFiers WilcoFiers modified the milestones: axe-core 4.6, Axe-core 4.7 Nov 30, 2022
@WilcoFiers WilcoFiers mentioned this issue Jan 19, 2023
Merged
@WilcoFiers WilcoFiers added the pr A pr has been created for the issue label Jan 19, 2023
@WilcoFiers WilcoFiers assigned WilcoFiers and unassigned Zidious Jan 19, 2023
@padmavemulapati
Copy link

Validated with the latest axe-core develop branch code base,
The Scrollable-region-focussible rule is not failing when select size makes scrollable for the list of items.
Test snippet:

<select size="2" name="select" disabled="disabled" aria-label="Test">
    <option value="First">First</option>
    <option value="Second">Second</option>
    <option value="Third">Third</option>
  </select>

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@WilcoFiers WilcoFiers

Labels
fix Bug fixes pr A pr has been created for the issue rules Issue or false result from an axe-core rule
Projects
None yet
Milestone
Axe-core 4.7
Development

Successfully merging a pull request may close this issue.

fix(scrollable-region-focusable): skip native controls dequelabs/axe-core
5 participants
@WilcoFiers @straker @MichaelKetting @Zidious @padmavemulapati