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

Footer issues #3124

Closed
6 tasks
thisisdano opened this issue Oct 3, 2019 · 4 comments
Closed
6 tasks

Footer issues #3124

thisisdano opened this issue Oct 3, 2019 · 4 comments

Comments

@thisisdano
Copy link
Member

thisisdano commented Oct 3, 2019

See https://designsystem.digital.gov/components/footer/

Screen Shot 2019-10-03 at 12 31 04 PM

  • We explicitly add a mobile-lg:grid-gap-0 to the contact info grid, but that seems to be a mistake.

  • If utilities aren't set to use important usa-footer__contact-info.grid-row will be overridden CSS to display: inline-block — we need to constrain this behavior to the slim footer only.

  • .usa-footer__logo-img needs an explicit width: 100%set

  • Big footer open carets are too big after svg refresh...

Screen Shot 2019-10-03 at 2 19 08 PM


  • Big footer should be able to accept nav items that don't contain sub-items, just as the primary nav can be a combination of simple links and links that trigger menus.

  • Use auto for default footer link spacing @desktop to improve horizontal spacing

Screen Shot 2019-10-03 at 3 35 58 PM

@thisisdano thisisdano changed the title Grid gap missing in slim footer contact info [@mobile-lg] Footer issues Oct 3, 2019
@mejiaj
Copy link
Contributor

mejiaj commented Oct 30, 2019

If utilities aren't set to use important usa-footer__contact-info.grid-row will be overridden CSS to display: inline-block — we need to constrain this behavior to the slim footer only.

Just to confirm this means:

.usa-footer__contact-info {
    display: inline-block;
    line-height: 1.1;
}

Should be:

.usa-footer__contact-info {
 ... all current styles minus `display: inline-block;`
}

.usa-footer--slim .usa-footer__contact-info {
  display: inline-block;
}

.usa-footer__logo-img needs an explicit width: 100%set

What issue is this causing and where? I assume this is an IE11 issue?

@thisisdano
Copy link
Member Author

It's not an IE 11 issue, rather an uncaught display mistake. For instance, if we look at the big footer, we see the contact info stacked (look in the bottom right...)
Screen Shot 2019-10-30 at 1 16 16 PM

It should appear as two columns:
Screen Shot 2019-10-30 at 1 16 27 PM

And yes, I think the fix is just as you indicate!

@thisisdano
Copy link
Member Author

For reference, all these issues came up as I was updating the uswds-jekyll theme and applying it to a reference site.

@thisisdano
Copy link
Member Author

Fixed in above PRs

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

No branches or pull requests

2 participants