fix(a11y): use distinct alt text for footer license icons#7971
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull request overview
Improves footer accessibility by preventing screen readers from announcing duplicate alternative text for the Creative Commons license icons, aligning with issue #7970.
Changes:
- Adds an
aria-labelto the/licensefooter link to provide a single accessible name. - Marks the two license icon images as decorative by setting
alt="".
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
alexander-akait
left a comment
alexander-akait
left a comment
There was a problem hiding this comment.
We should just add different alt for them
mr-baraiya
changed the title
|
Thanks for the feedback! I’ve updated the footer icons to use distinct alt text for each image. |
| src={BY} | ||
| width={25} | ||
| height={25} | ||
| /> |
There was a problem hiding this comment.
It should be Creative Commons License (CC) and Creative Commons Attribution (CC BY)
There was a problem hiding this comment.
Thanks for the suggestion! I've updated the footer icons with the requested alt text:
- Creative Commons License (CC)
- Creative Commons Attribution (CC BY)
Please let me know if anything else needs adjustment.
Fixes #7970
The footer license link contained two images with identical alt text
("Creative Commons License").
Screen readers may announce duplicated text.
This change provides distinct alt text for each Creative Commons icon
to avoid duplicate announcements by screen readers.