Conversation
rtwell
left a comment
rtwell
left a comment
There was a problem hiding this comment.
Looks great—my only (very minor and essentially irrelevant) request would be to swap in an ampersand in "privacy & security"
|
@rtwell changes made. |
There was a problem hiding this comment.
I believe this class exists already in Basscss.
There was a problem hiding this comment.
We can remove the py0.center.caps from this since it's no longer stacked on mobile and caps is already listed once.
There was a problem hiding this comment.
Do we want to actually link to the GSA website?
There was a problem hiding this comment.
I think that would probably be a good idea. It works well on the identity-site page.
There was a problem hiding this comment.
Add alt tags on both versions of GSA logo.
hursey013
left a comment
hursey013
left a comment
There was a problem hiding this comment.
Couple of small things, otherwise lgtm!
|
Squashed and merging! |
3 participants
This PR updates the visual appearance of the footer. From issue 18F/identity-private#2073
Large screens:
Small screens: