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

When using aria-activedescendant, the focused element is not scrolled into the visible area #2052

Closed
JAWS-test opened this issue Oct 4, 2021 · 6 comments · Fixed by #2056 or #2055
Closed
Labels
Example Page Related to a page containing an example implementation of a pattern Feedback Issue raised by or for collecting input from people outside APG task force QA for APG Examples Related to improve the quality and the acceptance of APG examples
Milestone

Comments

@JAWS-test
Copy link

Managing Focus in Composites Using aria-activedescendant say:

Move the visual focus indicator and, if necessary, scrolled the active element into view

Unfortunately, this is not even considered in the APG examples like https://w3c.github.io/aria-practices/examples/radio/radio-activedescendant.html, https://w3c.github.io/aria-practices/examples/menu-button/menu-button-actions-active-descendant.html and https://w3c.github.io/aria-practices/examples/combobox/combobox-select-only.html).

When do I notice this: e.g. when I zoom out to 320x265 px according to WCAG 1.4.10 and then navigate through the entries with the arrow keys. In some APG examples, the problem would occur even more often and even without zoom if there were more elements (e.g. radio buttons).

Related: #1135, w3c/wcag#2049 (comment)

@alastc
Copy link

alastc commented Oct 4, 2021

This seems like something that should be raised in the aria-practices issues? Or possibly as browser bugs.

@JAWS-test
Copy link
Author

This seems like something that should be raised in the aria-practices issues?

That's what it is

Or possibly as browser bugs.

I don't think so. If browsers are expected to focus aria-activedescendant correctly automatically, then there wouldn't be the sentence I quoted in ARIA APG that web designers have to do this

@alastc
Copy link

alastc commented Oct 4, 2021

Apologies, I get so many WCAG-issues emails, I didn't notice the change in repo. No wonder I couldn't tag it!

@mcking65 mcking65 added Example Page Related to a page containing an example implementation of a pattern Feedback Issue raised by or for collecting input from people outside APG task force QA for APG Examples Related to improve the quality and the acceptance of APG examples labels Oct 5, 2021
@mcking65
Copy link
Contributor

mcking65 commented Oct 5, 2021

@JAWS-test

This is an excellent observation.

@jesdaigle @smhigley Would it be possible to automate testing for visibility of elements given AT focus via aria-active descendant after changing view port size as part of our regression testing?

Given we have done code quality work on several of these examples recently, we should discuss fixes for this. Adding to agenda.

@a11ydoer
Copy link
Contributor

a11ydoer commented Oct 5, 2021

@jongund will look into this issue - APG meeting on Oct 5, 2021.

@a11ydoer
Copy link
Contributor

a11ydoer commented Oct 5, 2021

@jesdaigle will look into changing view port size as part of the regression testing. - APG meeting on Oct 5, 2021.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Example Page Related to a page containing an example implementation of a pattern Feedback Issue raised by or for collecting input from people outside APG task force QA for APG Examples Related to improve the quality and the acceptance of APG examples
Projects
None yet
4 participants