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

feat: Update banner aria props with changes from USWDS #2384

Merged
merged 3 commits into from
May 5, 2023

Conversation

brandonlenz
Copy link
Contributor

@brandonlenz brandonlenz commented May 4, 2023

Summary

Closes #2185

Related Issues or PRs

In response to uswds/uswds#4925 and uswds/uswds#4912, USWDS updated some aria attributes in https://github.com/uswds/uswds/pull/4925/files

How To Test

All CI checks pass

Verify aria attributes match USWDS source code

@brandonlenz brandonlenz requested a review from a user May 4, 2023 17:17
@brandonlenz brandonlenz changed the title Update banner aria props with changes from USWDS feat: Update banner aria props with changes from USWDS May 4, 2023
@brandonlenz brandonlenz enabled auto-merge (squash) May 5, 2023 20:14
@brandonlenz brandonlenz merged commit f8c359b into main May 5, 2023
@brandonlenz brandonlenz deleted the bl-update-banner-aria branch May 5, 2023 20:21
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.

Aria-hide lock icon and flag in banner
2 participants