Skip to content

Conversation

mahoneycm
Copy link
Contributor

@mahoneycm mahoneycm commented Aug 1, 2023

Summary

Changelog for
uswds/uswds#5413

Preview link

Range changelog →

Copy link
Contributor

@amyleadem amyleadem left a comment

Choose a reason for hiding this comment

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

Added a suggestion to include affectsMarkup and a note about the need to update markup to access changes.

Comment on lines 6 to 8
summary: Removed redundant ARIA attributes.
summaryAdditional: This ultimately improved accessibility usage, particularly with screen readers.
isBreaking: true
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
summary: Removed redundant ARIA attributes.
summaryAdditional: This ultimately improved accessibility usage, particularly with screen readers.
isBreaking: true
summary: Removed redundant ARIA attributes to improve the screen reader experience.
summaryAdditional: Update range markup to incorporate these updates.
isBreaking: true
affectsMarkup: true

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Made a small change to summaryAdditional because on first read, I didn't read it as instructions

summaryAdditional: To incorporate these changes, update your range component markup.

Let me know what you think!

Copy link
Contributor

@amyleadem amyleadem left a comment

Choose a reason for hiding this comment

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

Added one small suggestion to also include "affectsMarkup"

Copy link
Contributor

@amyleadem amyleadem left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@amyleadem amyleadem changed the base branch from main to release-3.7.0 October 27, 2023 20:52
@amyleadem amyleadem merged commit ee12f18 into release-3.7.0 Oct 31, 2023
@mejiaj mejiaj deleted the cm-5413-changelog branch February 21, 2024 15:47
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