669840 - Implement new and improved SUMO top header area

Status: VERIFIED FIXED

(support.mozilla.org :: General, defect)

Description

[David Tenser [:djst]]

Let's implement the more space-efficient and functional top header in SUMO. This is a follow-up bug for the actual implementation of bug 635265.

* Screenshot of how things should look like is here: attachment 542508 [details]
* Production file is here: http://intothefuzz.com/leetom/SUMO/PSD/sumo_header_01.psd

The only thing left to decide is the actual titles of the nav links. Michael, could you provide this information please?

Comment 1

[Verdi [:verdi] Best to slack me]

Desktop | Mobile | Firefox Home | Firefox Sync | Thunderbird

David, what do you think?

Comment 2

[David Tenser [:djst]]

Sounds good to me. Thanks!

Comment 3

[Ricky Rosario [:rrosario, :r1cky]]

(In reply to comment #1)
> Desktop | Mobile | Firefox Home | Firefox Sync | Thunderbird

This would be redundant with the Other Applications dropdown. See https://addons.mozilla.org/z/en-US/firefox/i/ for AMO's implementation.

Also, there are times when a page applies to multiple of these. For example, a sync article can be applicable to Desktop, Mobile, and Firefox Sync. The only page where there is a clear selection is on the landing pages for each product. Maybe we should we only show this sub nav on those pages?

Comment 4

[Verdi [:verdi] Best to slack me]

(In reply to comment #3)
> (In reply to comment #1)
> > Desktop | Mobile | Firefox Home | Firefox Sync | Thunderbird
> 
> This would be redundant with the Other Applications dropdown. See
> https://addons.mozilla.org/z/en-US/firefox/i/ for AMO's implementation.

I don't what that other applications menu is supposed to be for. We didn't get clarification in the design bug 635265

> 
> Also, there are times when a page applies to multiple of these. For example,
> a sync article can be applicable to Desktop, Mobile, and Firefox Sync. The
> only page where there is a clear selection is on the landing pages for each
> product. Maybe we should we only show this sub nav on those pages?

I'm not sure what you are asking. The content of each landing page can be configured manually. Some things will be on more than one landing page.

Comment 5

[Ricky Rosario [:rrosario, :r1cky]]

(In reply to comment #4)
I put the new header behind a flag so that it can be previewed and feedbacked before pushing it out to everybody. So once it lands (today or tomorrow hopefully) take a look and we'll go from there.

Comment 6

[Ricky Rosario [:rrosario, :r1cky]]

https://github.com/jsocol/kitsune/commit/2c222864cfafd3844d75f7e69adf45202654865d

New header is behind 'new-header' waffle flag for now to collect feedback and fix any blockers before rolling out to everybody.

Comment 7

[Verdi [:verdi] Best to slack me]

(In reply to comment #6)
> https://github.com/jsocol/kitsune/commit/
> 2c222864cfafd3844d75f7e69adf45202654865d
> 
> New header is behind 'new-header' waffle flag for now to collect feedback
> and fix any blockers before rolling out to everybody.

How can we see the new header?

Comment 8

[Ricky Rosario [:rrosario, :r1cky]]

(In reply to comment #7)
> How can we see the new header?

I just enabled it for everybody on stage https://support.allizom.org.

Please file new bugs for any issues or enhancements.

Comment 9

[Verdi [:verdi] Best to slack me]

Attachment: header changes

(In reply to comment #8)
> (In reply to comment #7)
> > How can we see the new header?
> 
> I just enabled it for everybody on stage https://support.allizom.org.
> 
> Please file new bugs for any issues or enhancements.

Noted changes on the attachement

Comment 10

[James Socol [:jsocol, :james]]

Let's file (a) follow-up bug(s) for those.

Comment 11

[Rebecca Billings [:rbillings]]

Verified new header looks like the original attachment, the nav links match on the page & in drop down