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

LG-4795: Change font to Public Sans #5900

Merged
merged 5 commits into from
Feb 15, 2022
Merged

LG-4795: Change font to Public Sans #5900

merged 5 commits into from
Feb 15, 2022

Conversation

aduth
Copy link
Contributor

@aduth aduth commented Feb 2, 2022

Why: As a user, I want to see a consistent font and brand across all login.gov sites, including the IdP, so that I can trust that this website is legitimate.

Before After
Screen Shot 2022-02-02 at 10 47 34 AM Screen Shot 2022-02-02 at 10 48 06 AM

Copy link
Contributor

@mitchellhenke mitchellhenke left a comment

Choose a reason for hiding this comment

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

👏🏼 👏🏼 👏🏼 👏🏼 👏🏼 👏🏼

@aduth aduth force-pushed the aduth-lg-4795-public-sans branch from 40dbe46 to 7fc758f Compare February 7, 2022 13:25
@aduth aduth marked this pull request as ready for review February 7, 2022 13:26
@aduth aduth force-pushed the aduth-lg-4795-public-sans branch from 3ac0165 to b09c835 Compare February 14, 2022 15:01
**Why**: Avoid excess wrapping, match Figma designs
* Match font size between password strength meter label and strength

https://docs.google.com/document/d/1H2sGTfajE_vFywuS-dErrveE3HYeraOpKqBZoU7zW40/edit#bookmark=id.ij7e80fv4x9p

* Fix line-height to match body content for heading override classes

https://docs.google.com/document/d/1H2sGTfajE_vFywuS-dErrveE3HYeraOpKqBZoU7zW40/edit#bookmark=id.cjccn2jqw5or

* Improve account history text wrapping

https://docs.google.com/document/d/1H2sGTfajE_vFywuS-dErrveE3HYeraOpKqBZoU7zW40/edit#bookmark=id.s69rw07kvjb8

* Inherit h3 styles for Troubleshooting Options heading

https://docs.google.com/document/d/1H2sGTfajE_vFywuS-dErrveE3HYeraOpKqBZoU7zW40/edit#bookmark=id.ar9ig16cx9yv

* Reduce padding on IAL2 verify accordion content

https://docs.google.com/document/d/1H2sGTfajE_vFywuS-dErrveE3HYeraOpKqBZoU7zW40/edit#bookmark=id.97mep966gkz4

* Add grid gap for IAL2 verify details

https://docs.google.com/document/d/1H2sGTfajE_vFywuS-dErrveE3HYeraOpKqBZoU7zW40/edit#bookmark=id.1s7zqgip92ek

* Fix footer wrap at small viewports

https://docs.google.com/document/d/1H2sGTfajE_vFywuS-dErrveE3HYeraOpKqBZoU7zW40/edit#bookmark=id.ae3a6eue3vf2

* Fix account mobile header wrapping

https://docs.google.com/document/d/1H2sGTfajE_vFywuS-dErrveE3HYeraOpKqBZoU7zW40/edit#bookmark=id.s0lnbwisvwow

* Fix email wrapping on account page

* Wrap phone number default on mobile

https://docs.google.com/document/d/1H2sGTfajE_vFywuS-dErrveE3HYeraOpKqBZoU7zW40/edit#bookmark=id.st157qhckyoi

* Move IAL2 sign-in PII accordion content up

https://docs.google.com/document/d/1H2sGTfajE_vFywuS-dErrveE3HYeraOpKqBZoU7zW40/edit#bookmark=id.yariyhzc6hsn

* Fix typo on MFA select screen

https://docs.google.com/document/d/1H2sGTfajE_vFywuS-dErrveE3HYeraOpKqBZoU7zW40/edit#bookmark=id.jj68sbwse7pe

* Collapse excess whitespace on webauthn setup

https://docs.google.com/document/d/1H2sGTfajE_vFywuS-dErrveE3HYeraOpKqBZoU7zW40/edit#bookmark=id.hgc1uwxpfono

* Adjust vertical spacing for reauthn password

https://docs.google.com/document/d/1H2sGTfajE_vFywuS-dErrveE3HYeraOpKqBZoU7zW40/edit#bookmark=id.mveu2u207pjz

* Replace b with span.text-bold

See: #5937 (comment)

* Update specs

* Fix positioning of success icon in review accordion

* Increase margin above IAL2 review password field

* Move IAL2 sign-in description outside footer
@aduth aduth force-pushed the aduth-lg-4795-public-sans branch from 2c797ea to c3cd821 Compare February 15, 2022 16:10
@aduth aduth merged commit ec48b33 into main Feb 15, 2022
@aduth aduth deleted the aduth-lg-4795-public-sans branch February 15, 2022 16:40
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.

2 participants