-
Notifications
You must be signed in to change notification settings - Fork 438
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
Comments
Thank you! I am sure you will add the text to translation file as well. |
@reetagithub : Yes, the merged PR already makes it translatable. See the updates to the |
Perfect!
I hope that the texts to be translated will be put to the files, especially fi.json5. I tried to update it myself previously this year, and it became a mess, so now I will wait for some one else (who knows how to) to update the translation files.
Thank you, Tim!
Tim Donohue ***@***.***> kirjoitti 13.11.2023 kello 17.19:
@reetagithub<https://github.com/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
—
Reply to this email directly, view it on GitHub<#2523 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AMIY5JFFEXRFCP5757H3JNDYEI3B3AVCNFSM6AAAAAA5HIKCNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBYGM3DKMRSG4>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
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. |
@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 |
@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. |
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.
The text was updated successfully, but these errors were encountered: