Skip to content

Conversation

amyleadem
Copy link
Contributor

@amyleadem amyleadem commented Sep 7, 2023

Summary

Add changelog entry for identifier screen reader copy edit.

Related PR

uswds/uswds#5491

Preview link

Identifier changelog

@amyleadem amyleadem changed the base branch from main to release-3.7.0 October 27, 2023 21:31
@amyleadem amyleadem marked this pull request as ready for review October 27, 2023 22:07
@amyleadem amyleadem requested review from mejiaj and mahoneycm October 27, 2023 22:07
Copy link
Contributor

@mahoneycm mahoneycm left a comment

Choose a reason for hiding this comment

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

Lookin' good! One small optional change.

@@ -2,6 +2,14 @@ title: Identifier
type: component
changelogURL:
items:
- date: NNNN-NN-NN
summary: Updated the screen reader readout to say "Official" instead of "An official".
summaryAdditional: Users should update their markup to improve the screen reader experience.
Copy link
Contributor

Choose a reason for hiding this comment

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

Optional change

Should we highlight why we made the change? Something like:

Suggested change
summaryAdditional: Users should update their markup to improve the screen reader experience.
summaryAdditional: This change prevents screen reader users from mishearing "Unofficial" instead of "An official". Users should update their markup to improve the screen reader experience.

Feel free to change the language.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good note. I updated it to be more similar to both the release notes and the related banner changelog (issue #5925).

Copy link
Contributor

@mahoneycm mahoneycm left a comment

Choose a reason for hiding this comment

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

Looks great!

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.

2 participants