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.

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

scrollable-region-focusable rule false positive on clip overflow #3283

Closed
backwardok opened this issue Nov 12, 2021 · 2 comments
Closed

scrollable-region-focusable rule false positive on clip overflow #3283

backwardok opened this issue Nov 12, 2021 · 2 comments
Labels
fix Bug fixes pr A pr has been created for the issue rules Issue or false result from an axe-core rule
Milestone

Comments

@backwardok
Copy link

Product: axe Extension (not sure if it exists elsewhere as well, but found on the axe extension)

Expectation: Setting overflow to clip doesn't get flagged as an error for scrollable-region-focusable since it doesn't actually scroll

Actual: Setting overflow to clip gets flagged as an error for scrollable-region-focusable

Motivation: Results in false positives, which encourages people to turn off the rule or ignore the results

Example codepen to test behavior

axe extension version: 4.3.5
Browser version: Chrome 95.0.4638.69
Platform: Mac

Similar issue to #1582

@WilcoFiers
Copy link
Contributor

Seems I forgot to comment... apologies! Thanks for reporting this. I confirmed the issue, and will have a fix for it in the next release.

@WilcoFiers WilcoFiers added fix Bug fixes rules Issue or false result from an axe-core rule labels Nov 19, 2021
@WilcoFiers WilcoFiers added this to the Axe-core 4.4 milestone Nov 19, 2021
@WilcoFiers WilcoFiers added the pr A pr has been created for the issue label Nov 24, 2021
@padmavemulapati
Copy link

Verified with the latest dev branch code , not seeing failure on scrillable-region-focusible when clip as hidden.
This is reproducible in the old code using axe-core3.4.1

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

No branches or pull requests

3 participants