1108810 - Implement footer for new IA

Status: RESOLVED FIXED

(Webmaker Graveyard :: webmaker.org, defect)

Description

[Bobby Richter [:secretrobotron]]

Design:
- large: https://redpen.io/hsc2386be2e3ff93eb
- small: https://redpen.io/fk3aeecca7291ac443

Details:
- on /, /explore, /tools, /about
- About -> webmaker.org/about
- Teach -> teach.webmaker.org
- Donate -> sendto.mozilla.org
- Blog -> blog.webmaker.org
- Contact -> [e-mail] (this one isn't in the design; just add it under Blog)
- Get help -> support.mozilla.org/en-US/products/webmaker
- Privacy -> webmaker.org/privacy
- Legal -> webmaker.org/terms

Comment 1

[Aki Rose Braun [:akirose]]

Attachment: https://github.com/mozilla/webmaker.org/pull/1023

Comment 2

[Bobby Richter [:secretrobotron]]

Good to resolve?

Comment 3

[Aki Rose Braun [:akirose]]

I'll leave that final decision up to Matthew

Comment 4

[Matthew Willse [:mattheww]]

One easy change. Sorry that I posted this to the header IA bug. 

Neglected by my comp: In the footer, the CTA and footer nav occupy the same box of soup. To match Ricardo's splash page redesign and anchor the footer nav, let's add:

.navbar-footer { 
 background:#01527d; /* @lightnavy */
 padding-top:1.5em;
}

Beyond that I R+ the footer and hand this over to whoever is on for Quality.

Comment 5

[Aki Rose Braun [:akirose]]

Marking this as done as it's been rolled up to the meta-bug #1104272

Comment 6

[Aki Rose Braun [:akirose]]

Attachment: https://github.com/mozilla/webmaker.org/pull/1035

Comment 7

[Gavin Suntop [:gvn]]

Comment on attachment 8536897 [details] [review]
https://github.com/mozilla/webmaker.org/pull/1035

LGTM!

Comment 8

[Bobby Richter [:secretrobotron]]

Able to close/land?

Comment 9

[Aki Rose Braun [:akirose]]

While the footer work isn't done, it's being wrapped up in bug #1111822 so yeah we can close this one.

Comment 10

[[github robot]]

Commits pushed to master at https://github.com/mozilla/webmaker.org

https://github.com/mozilla/webmaker.org/commit/b9db7befc7c33a8628a37c7ca9c570221c409e92
Fixes Bug 1108810 - Implement footer for new IA

https://github.com/mozilla/webmaker.org/commit/34b7582d7294d529fa2400fddec62296cc114772
Merge pull request #1035 from gesa/1108810/footer

fixes bug #1108810

https://github.com/mozilla/webmaker.org/commit/5953f30827aca7c225b92896882ea59787f70c88
Fixes Bug 1108810 - Implement footer for new IA