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 to main content link is not present in the user pages #2523

Closed
reetagithub opened this issue Sep 26, 2023 · 6 comments · Fixed by #2607
Closed

Skip to main content link is not present in the user pages #2523

reetagithub opened this issue Sep 26, 2023 · 6 comments · Fixed by #2607

Comments

@reetagithub
Copy link
Contributor

Is your feature request related to a problem? Please describe.
This is an accessibility issue, an important feature is missing from DSpace7. When using a screen reader, the user should have the opportunity to skip all the stuff in the upper part of the screen (navigation, login, logo etc.) and get to the main contents of the page. This is a requirement in the accessibility legislation in the EU which is based on WCAG 2.2 levels A and AA. This More info here (Bypass Blocks) and here Webaim.

Describe the solution you'd like
In the beginning of each page there should be a hidden link that leads to the main content of the page and skips the irrelevant content. You can see a way to do this right on WebAim's page and its source code ('skiptocontent')

Describe alternatives or workarounds you've considered
There is no workaround. This feature is crucial for the users with screen readers, and it is part of accessibility requirements. If this is not made, it is very difficult for vision impaired users to explore the pages.

@reetagithub reetagithub added needs triage New issue needs triage and/or scheduling new feature labels Sep 26, 2023
@github-project-automation github-project-automation bot moved this to 🆕 Triage in DSpace Backlog Sep 26, 2023
@tdonohue tdonohue added accessibility high priority bug help wanted Needs a volunteer to claim to move forward and removed needs triage New issue needs triage and/or scheduling labels Sep 26, 2023
@tdonohue tdonohue removed the help wanted Needs a volunteer to claim to move forward label Nov 8, 2023
@tdonohue tdonohue moved this from 📋 To Do to 🏗 In Progress in DSpace 8.x and 7.6.x Maintenance Nov 8, 2023
@tdonohue tdonohue added this to the 7.6.1 milestone Nov 10, 2023
@github-project-automation github-project-automation bot moved this from 🏗 In Progress to ✅ Done in DSpace 8.x and 7.6.x Maintenance Nov 10, 2023
@reetagithub
Copy link
Contributor Author

Thank you! I am sure you will add the text to translation file as well.

@tdonohue
Copy link
Member

@reetagithub : Yes, the merged PR already makes it translatable. See the updates to the en.json5 file: https://github.com/DSpace/dspace-angular/pull/2607/files#diff-63dcd993feb69f5096737902df76b82a423ebd1acc78cc9b37798b2c05a5436f

@reetagithub
Copy link
Contributor Author

reetagithub commented Nov 14, 2023 via email

@anis-moubarik
Copy link

This feature doesn't seem to work on Firefox, I can access the button, but clicking it does nothing.

Also with Chrome and Firefox, while logged in, the button is obstructed by the left side panel.

@alexandrevryghem
Copy link
Member

@anis-moubarik: I just tested it again for Firefox & Chrome and clicking on it & hitting enter on it does work. The next time you click on the tab the focus will be on the leading institutions using DSpace url and not to admin sidebar/header. When you are logged in I can indeed see that the admin sidebar is displayed over it, we should probably check if this is still happening after #2676 is merged in

@anis-moubarik
Copy link

@alexandrevryghem You're right! I'm used to the viewport centering the content with jump to content -links (like it seems to do with Chrome natively). I'll probably add scrollIntoView call in the skipToMainContent function.

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

Successfully merging a pull request may close this issue.

4 participants