Add aria-label to repeated source card links - #55
Open
claude[bot] wants to merge 1 commit into
Open
Conversation
Each project card had an identical visible "source" link with no distinguishing accessible name, violating WCAG 2.4.4 Link Purpose (In Context), Level A. Added aria-label with the project name to each link so screen reader users can tell them apart. Fixes #47
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
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.
Fixes #47
What changed
Added
aria-labelattributes to the four project card "source" links indemo/index.html:aria-label="dotfile-sync source code on GitHub"aria-label="minimal-tab source code on GitHub"aria-label="deploy-pipe source code on GitHub"aria-label="weather-api source code on GitHub"Visible link text remains "source" — only the accessible name changes.
Why
Screen reader users navigating by link list encountered four identical "source" entries with no way to distinguish which project each referred to. This violated WCAG 2.4.4 Link Purpose (In Context), Level A.
How to verify
Open
demo/index.htmlwith a screen reader (e.g. NVDA + Chrome or VoiceOver + Safari) and use the links list shortcut. Each "source" entry should now read the project name in addition to "source".