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

'skip navigation' button should become visible when it has focus #3040

Closed
moloko opened this issue Feb 17, 2021 · 1 comment · Fixed by #3062
Closed

'skip navigation' button should become visible when it has focus #3040

moloko opened this issue Feb 17, 2021 · 1 comment · Fixed by #3062

Comments

@moloko
Copy link
Contributor

moloko commented Feb 17, 2021

At least, according to the page about this accessibility feature in Techniques for WCAG 2.0:

Check that the link is either always visible or visible when it has keyboard focus

I'm sure at one point it did display when it had focus but this functionality seems to have been lost/changed at some point.

@moloko
Copy link
Contributor Author

moloko commented Feb 17, 2021

Also just to note it has a class a11y-ignore-focus attached to it but I can only find one other reference to this, here:

a11y_wrap_focus() {
const cfg = Adapt.config.get('_accessibility');
if (cfg._isPopupWrapFocusEnabled === false) return '';
return new Handlebars.SafeString('<a class="a11y-focusguard a11y-ignore a11y-ignore-focus" role="presentation">&nbsp;</a>');
},

Is this something 'legacy' that needs to be removed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant