-
Notifications
You must be signed in to change notification settings - Fork 12
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
Update to USWDS v3.9.0 #1397
Update to USWDS v3.9.0 #1397
Conversation
)} | ||
{!slim && <slot name="headline"></slot>} | ||
<slot></slot> | ||
</div> |
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.
I have added a wrapping div
around the slot content because the USWDS alert module styles the alert content with the following CSS:
.usa-alert--info .usa-alert__body > * {
margin-left: 2.5rem;
}
The selector > *
will not work on a placeholder element like <slot>
when it's the direct descendent.
The alert content is displaying like this without that div
:
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.
This is happening now because this update was made:
Fixed a bug that caused $theme-site-margins-width to unexpectedly adjust the alignment inside the alert and site alert components. Alignment on the alert and site alert components will likely shift from this change. Confirm that your implementation of the component aligns as expected.
.github/workflows/chromatic.yml
Outdated
@@ -32,6 +32,6 @@ jobs: | |||
token: ${{ secrets.GITHUB_TOKEN }} | |||
# 👇 Chromatic projectToken, refer to the manage page to obtain it. | |||
projectToken: ${{ secrets.CHROMATIC_PROJECT_TOKEN }} | |||
onlyChanged: true | |||
onlyChanged: false |
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.
Will revert before merging.
We already have a ticket created for the lock icon alignment: |
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.
That is quite the difference of padding on the alert. Design may need to update figma.
Looks good to me!
Chromatic
https://3401-uswds-390--65a6e2ed2314f7b8f98609d8.chromatic.com
Description
What's new in USWDS 3.9.0: uswds/uswds#6098
va-alert before
va-alert after
va-pagination before
va-pagination after
This version updates the number links with an underline:
Closes department-of-veterans-affairs/vets-design-system-documentation#3401
QA Checklist
Acceptance criteria
Definition of done