Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Accessibility fixes and other small code changes #4781

Merged
merged 5 commits into from
Aug 13, 2024

Conversation

Copy link
Contributor

Thanks Jaylyn-Barbee for opening a Pull Request! The reviewers will test the PR and highlight if there is any conflict or changes required. If the PR is approved we will proceed to merge the pull request 🙌

@Jaylyn-Barbee Jaylyn-Barbee changed the title Accessibility fixes and other small code changes [Do not merge] Accessibility fixes and other small code changes Aug 8, 2024
@Jaylyn-Barbee Jaylyn-Barbee changed the title [Do not merge] Accessibility fixes and other small code changes Accessibility fixes and other small code changes Aug 8, 2024
@Jaylyn-Barbee Jaylyn-Barbee added this pull request to the merge queue Aug 13, 2024
Copy link
Contributor

@jgw96 jgw96 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks Jaylyn. This is a perfect use of classMap

Merged via the queue into staging with commit 6bfdc6d Aug 13, 2024
6 of 8 checks passed
@Jaylyn-Barbee Jaylyn-Barbee deleted the jay/accessibility-0808 branch August 13, 2024 17:20
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