Header responsive menu styles#582
Conversation
|
2099795 to
e9c9701
Compare
Basic Header
Confirm:At desktop:
At mobile:
Mock-ups
Completed itemsThese items may have been completed prior to this PR.
|
Desktop review:
|
Should be the one in this folder: https://github.com/cfpb/design-system-react/tree/main/src/assets/images |
|
@flacoman91 I will provide updated mock-ups for the header with navigation (desktop and mobile) next week. |
e9c9701 to
973cf38
Compare
e0445bc to
980d929
Compare
natalia-fitzgerald
left a comment
There was a problem hiding this comment.
@flacoman91
The Basic Header implementation is looking good. Can you confirm what changed about the padding of the US gov banner at mobile & desktop widths?
The US gov banner wasn't getting the correct padding on the left side at mobile. It was 30px, and should drop down to 15px at <901px. It's used for the header, so I had to fix it to get the flag to align with the cfpb logo
The fix in this PR:
|






Updates header, pulls work out of #560
Fixes in this PR include:
Header
Banner
To test, go to preview, look at:
Screenshots:
