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

Accesibility: missing "jump to content" link #1117

Closed
jonpincus opened this issue Jun 8, 2023 · 3 comments · Fixed by #1158
Closed

Accesibility: missing "jump to content" link #1117

jonpincus opened this issue Jun 8, 2023 · 3 comments · Fixed by #1158
Labels
bug Something isn't working

Comments

@jonpincus
Copy link

jonpincus commented Jun 8, 2023

Issue Summary

There's no "skip to content" link allowing users with screenreaders to skip over all the headers.

Steps to Reproduce

Go to the main page of any lemmy site. Click tab. It should take you to a "jump to content" link, but none appears.

The screenshot of Reddit's interface illustrates what should happen. The "jump to content" link in the image is initally invisible, but appears when I hit a tab.

Screen Shot 2023-06-08 at 11 37 17 AM

Technical details

https://www.useragentman.com/blog/2016/12/15/implementing-an-accessible-skip-navigation-link-requires-more-thought-than-youd-think/ is a good discussion of how to provide this functionality (although a little old, not sure if best practices have evolved since then)

@jonpincus jonpincus added the bug Something isn't working label Jun 8, 2023
@skspade
Copy link
Contributor

skspade commented Jun 10, 2023

I'm willing to pick this up @SleeplessOne1917 is this something you all would want?

@SleeplessOne1917
Copy link
Member

If @dessalines and @Nutomic are good with it, it's fine by me. Accessibility is never a bad thing.

@skspade
Copy link
Contributor

skspade commented Jun 10, 2023

Agreed, I'll pick this up!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants