-
Notifications
You must be signed in to change notification settings - Fork 1.1k
USWDS 3.4.0 #5170
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
Merged
Merged
USWDS 3.4.0 #5170
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: James Mejia <james.mejia@gsa.gov>
Co-authored-by: James Mejia <james.mejia@gsa.gov>
- Use code for pipe character - Add empty alt text for pseudoelement to prevent character being read on screenreaders Based on guidance https://www.w3.org/TR/css-content-3/#alt
Sync develop 3.3.0
USWDS: Modal with nested forms. Resolves #4953
Since the compatibility choice is binary (support older browsers or only modern browsers), we don't need granularity in our font key. We only need "output all formats" or "output only modern formats". This simplifies the code to have a simple `true`/`false` for a `compatibility` key
Output woff and ttf font-face references only with explicit compatibility setting
mejiaj
approved these changes
Mar 9, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! I've tested:
- Ran
npm audit
and verified security info - Ran
npm test
without issues - Compiled SASS (
npx gulp compileSass
) successfully - Built templates (
npm run build:html
) successfully - Reviewed release notes for spelling errors
- Looked for old USWDS references
Ugh, I did find an old |
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What’s new in USWDS 3.4.0
Features and bug fixes
usa-banner
usa-layout-grid
package withusa-banner
resulting in a much smaller package size. (#5000)uswds-core
woff2
webfont format with our default settings. Since we no longer support IE11 and other old browsers, we no longer need to serve thewoff
andttf
formats. Teams that need this support can activate it with a new setting:$theme-font-browser-compatibility: true
. This is only a breaking change if your project needs to support IE11. (#5108)usa-header
"none"
for anymax-width
setting, including$theme-header-max-width.
(#5072)usa-in-page-navigation
usa-alert
uswds-core
1.5
for$theme-alert-padding-y
. Thanks @aduth! (#5076)usa-banner
usa-footer
.usa-footer__primary-link
without it reverting to anh4
when the JavaScript rebuilds the element for mobile. Thanks @paulathevalley! (#5044)usa-header
usa-link
usa-modal
usa-step-indicator
uswds-utilities
uswds-core
05
token. Now, the.outline-05
utility class and theu-outline('05')
mixin should work as expected. (#5079)Dependencies and security
gulp-dart-scss
gulp-mocha
mocha
sass-embedded
snyk
0
vulnerabilities in regular dependencies (dependencies for USWDS projects installed withnpm install uswds
)1 moderate 33 high
vulnerabilities in devDependencies (development dependencies)Release TGZ SHA-256 hash:
7b0336d95fa6e9f31b27a27424f3a53e84b415e09b5bbff5d9ec8a398fb406d7