Skip to content

Conversation

@aloverso
Copy link
Contributor

@aloverso aloverso commented Apr 4, 2025

Link to pivotal/JIRA issue

newjersey/accessibility-pm#16

Is PM acceptance required? (delete one)

  • No - merge after code review approval

What was done?

Add empty alt text to left-arrow pseudo-element on back button, so that it is flagged as decorative content and not read aloud by screen readers

How to test?

use voiceover or similar

Screenshots (use audio)

BEFORE:

Screen.Recording.2025-04-03.at.13.59.19.mov

AFTER:

Screen.Recording.2025-04-04.at.10.38.57.mov

@github-actions
Copy link

github-actions bot commented Apr 4, 2025

Heroku app: https://gyr-review-app-5869-1fb638f2cde3.herokuapp.com/
View logs: heroku logs --app gyr-review-app-5869 (optionally add --tail)

a {
&:before {
content:'';
content:'' / '';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[dust] - this syntax is new to me, and is cool! looked it up and here is a link in case anyone is interested:

https://www.w3.org/TR/css-content-3/#alt

@aloverso aloverso merged commit 38d5439 into main Apr 7, 2025
7 checks passed
@aloverso aloverso deleted the nj-316-a11y-back-arrow-label branch April 7, 2025 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants