Fix #999 Add skip to content link on Home, Slide and Info Pages for accessibility#1055
Open
chandrikarj wants to merge 3 commits intocamicroscope:developfrom
Open
Fix #999 Add skip to content link on Home, Slide and Info Pages for accessibility#1055chandrikarj wants to merge 3 commits intocamicroscope:developfrom
chandrikarj wants to merge 3 commits intocamicroscope:developfrom
Conversation
birm
requested changes
Jan 23, 2025
Member
birm
left a comment
birm
left a comment
There was a problem hiding this comment.
I'm not seeing this button consistently when I test this, see below with similar results on other pages.
Screen.Recording.2025-01-22.at.10.42.37.PM.mov
Contributor
Author
|
Hi @birm, The button is normally hidden but appears when the user presses the Tab key for the first time after landing on the page. It is hidden again after the user presses on it and skips to the content of the page. After that, it will show up again if the user navigates through all the links in the content using the Tab key. This behavior is same on the Slides and Info pages. If you'd prefer to have it displayed permanently, I can make that change. Thanks! Screen.Recording.2025-01-25.at.8.16.10.PM.mov |
|
This PR is stale because it has been open 45 days with no activity. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add skip to content link on Home, Slide and Info Pages for easier accessibility and better navigation
Summary
This pull request adds a "Skip to Content" link to the Home, Slides, and Info pages to improve accessibility. The link allows users to bypass the navigation menu and directly focus on the main content, enhancing the experience for keyboard and screen reader users.
Key changes include:
Added a visually styled "Skip to Content" link at the top of the element.
Ensured the link points to the appropriate id of the main content in each page.
Styled the link for visibility when focused and for a clean appearance.
Screenshots:
after-caMicroscope.mov
GIF of the changes:
Motivation
This change addresses accessibility issues highlighted in #999 . It ensures the website adheres to accessibility standards and improves navigation for users relying on keyboards or assistive technologies.
Testing
Verified functionality on Home, Slides, and Info pages by:
Pressing Tab to focus on the "Skip to Content" link.
Observing that pressing Enter moves the focus directly to the main content.
Tested styling changes to ensure the link is visually unobtrusive but noticeable when focused.
Verified no disruptions to existing functionality.
Questions
Any feedback or suggestions for improvement are welcome.