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: updates for USWDS v3.7 #2680

Merged
merged 13 commits into from
Dec 28, 2023
Merged

feat: updates for USWDS v3.7 #2680

merged 13 commits into from
Dec 28, 2023

Conversation

werdnanoslen
Copy link
Contributor

@werdnanoslen werdnanoslen commented Dec 27, 2023

Summary

Resolves #2682

The following changes in the 3.7 and 3.7.1 updates involved more than just scss updates. Changes that are checked off have been committed to this PR; the others are available for others to contribute updates to (or remove if no update is needed):

Related Issues or PRs

How To Test

Screenshots (optional)

@werdnanoslen werdnanoslen changed the title Updates for USWDS v3.7 chore: updates for USWDS v3.7 Dec 27, 2023
@brandonlenz brandonlenz changed the title chore: updates for USWDS v3.7 feat: updates for USWDS v3.7 Dec 27, 2023
@brandonlenz
Copy link
Contributor

Made feat since we want this to show in the changelog

Copy link
Contributor

github-actions bot commented Dec 27, 2023

Warnings
⚠️

Changes were made to package.json, but not to yarn.lock - Perhaps you need to run yarn install?

Generated by 🚫 dangerJS against 75bf391

@brandonlenz
Copy link
Contributor

Added Resolves #2682 to PR body

@werdnanoslen werdnanoslen marked this pull request as ready for review December 27, 2023 23:05
@werdnanoslen werdnanoslen requested review from a team as code owners December 27, 2023 23:05
Comment on lines +135 to +136
<span aria-hidden="true">An</span>
{` official website of the `}
Copy link
Contributor

Choose a reason for hiding this comment

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

Yay no more screen reader Unofficial vs An official silliness!

Copy link
Contributor

@brandonlenz brandonlenz left a comment

Choose a reason for hiding this comment

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

I recommend making the change from defaultValue to value in the RangeInput specifically.

Everything else looks good. Some other optional suggestions to go along with the change I'm interested in

src/components/Modal/Modal.tsx Outdated Show resolved Hide resolved
src/components/Modal/Modal.test.tsx Outdated Show resolved Hide resolved
src/components/forms/DatePicker/i18n.ts Show resolved Hide resolved
src/components/forms/RangeInput/RangeInput.tsx Outdated Show resolved Hide resolved
src/components/forms/RangeInput/RangeInput.tsx Outdated Show resolved Hide resolved
src/components/forms/RangeInput/RangeInput.tsx Outdated Show resolved Hide resolved
@brandonlenz
Copy link
Contributor

Thanks for making those changes!!

@werdnanoslen werdnanoslen merged commit 9a06e7d into main Dec 28, 2023
8 checks passed
@werdnanoslen werdnanoslen deleted the an-3.7updates branch December 28, 2023 18:00
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.

[feat] Update to USWDS 3.7.x
3 participants