-
Notifications
You must be signed in to change notification settings - Fork 16
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
Upgrade USWDS from v3.4.1 to v3.6.0 #362
Conversation
2c4cf25
to
f591c80
Compare
Checks aren't triggering 🤔 Will try closing and reopening. |
3.6.0 changed some details around disabled colors which is triggering the visual regression failure and inverting the disabled button color. I'll need to investigate. Before: https://federalist-340d8801-aa16-4df5-ad22-b1e3e731098e.sites.pages.cloud.gov/site/18f/identity-design-system/buttons/
|
Setting 'disabled-dark' as white will probably get us in trouble, we should revisit the disabled colors
@nickttng When you have a moment, I might want to tag you in here to weigh in on a couple things:
|
Anti-aliasing Disabled button colors |
@aduth When it's merged, I'll update the disabled buttons in the Figma library. |
I traced it back to the work here: uswds/uswds#5250 Related issue: uswds/uswds#5180 They mention one of the reasons is it being unsupported: https://developer.mozilla.org/en-US/docs/Web/CSS/font-smooth I'm not really compelled by the case, though I also wonder if it's a battle worth fighting, considering that everything seems stacked against the use of this style. I'm also not sure how much effort it'd be to bring back for all the places it may have been removed, so I'll take a look. |
Couple updates here:
I tried to approximate this as close as possible with our "grey" color family, but it's somewhat lighter than USWDS. That's because USWDS uses @nickttng Any thoughts here? Personally I think it could work as-is.
I'm feeling like it's a losing battle to try to keep the font-smoothing:
One possible compromise I'm wondering is if we should consider changing some bold font weights to normal. I was actually surprised that button text is bolded by default (from USWDS)? Maybe overriding that to normal font weight could make the change less noticeable.
Obviously this just applies to buttons, but maybe (a) that's the most "problematic", and (b) are there others we could do similar? e.g. navigation current link. @nickttng Are you okay with dropping the font smoothing? Any interest in the idea above to override font weight for buttons? |
@aduth Thanks for the context
Yes, it works as-is
Yes, okay with dropping the font smoothing. The normal w/o font smoothing appears to be a better approach than the bold w/o font smoothing. My concern with the normal version for the What do you think about proceeding with what USWDS has established? And we would keep an eye out via user feedback / usability studies? |
If you mean (1) dropping font smoothing and (2) keeping the default bold text, then yes, I'm okay with that, and it simplifies things to stay in alignment with USWDS 👍 In which case, all of that should be reflected in the current preview branch, so I think we're good to move forward if we're okay with everything there? |
Yep, with 1) dropping font smoothing and 2) keeping the default bold text. 👍🏼 |
I may hold off temporarily on publishing a new version, since I believe a new patch release of USWDS should be landing soon. Their monthly newsletter from 10 days ago mentioned a 3.6.1 release on the horizon. |
Upgrades USWDS to its latest version, from 3.4.1 to 3.6.0.
Release notes:
Also updates
dependabot.yml
to change the dependency name (the npm package was renamed fromuswds
to@uswds/uswds
in v3.0.0), since the purpose of the configuration is to automate these updates.Blocked: There's a build issue that needs to be sorted out. Also reported upstream at uswds/uswds#5334 .Edit: This is now fixed in USWDS 3.6.0.