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

Accessibility issue: invisible skip link #1088

Open
scoutb-cogapp opened this issue Aug 13, 2024 · 2 comments
Open

Accessibility issue: invisible skip link #1088

scoutb-cogapp opened this issue Aug 13, 2024 · 2 comments

Comments

@scoutb-cogapp
Copy link

UV version: [email protected]

I'm submitting a: bug report => please fork one of these codesandbox examples with a repro of your issue and include a link to it below

Page area

"skip to downloads" link at top of page

Issue description

There is a link on top of the page enabling users to skip to the downloads and alternative formats section at the bottom of the image viewer. When this link is accessed by keyboard it remains invisible. A keyboard user won't know where focus has moved.

Steps to reproduce

  1. open this manifest
  2. tab to the first item in the tab order (first item button), then tab backwards
  3. observe that no link has become visible and no focus highlight is applied to any element

Expected behaviour

The link becomes visible and receives focus when it is accessed by keyboard

WCAG criterion

2.4.7 Focus Visible (Level AA)

Related code

<a class="sr-only" href="#download-btn">Skip to downloads and alternative formats</a>
@damodeburca
Copy link

@demiankatz Looks good to me 👍and the link brings the user to the download button with focus. Perhaps as a separate discussion we could agree the order of focus as the user tabs with the keyboard - currently it goes to the first item 'box' - I guess thats correct.

@demiankatz
Copy link
Contributor

Thanks, @damodeburca. Regarding focus order, there are quite a few other open issues addressing various aspects of the order; it's probably simplest to resolve this first and then discuss further as we work through those other items.

demiankatz added a commit to FalveyLibraryTechnology/universalviewer that referenced this issue Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Community Sprint IN TESTING
Development

No branches or pull requests

4 participants