Implement new and improved SUMO top header area

VERIFIED FIXED in 2011-08-02

Status

VERIFIED FIXED
8 years ago
8 years ago

People

(Reporter: djst, Assigned: rrosario)

Tracking

unspecified
2011-08-02

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
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

8 years ago
Desktop | Mobile | Firefox Home | Firefox Sync | Thunderbird

David, what do you think?
(Assignee)

Updated

8 years ago
Target Milestone: --- → 2011Q3
(Reporter)

Comment 2

8 years ago
Sounds good to me. Thanks!
(Assignee)

Comment 3

8 years ago
(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?
(Assignee)

Updated

8 years ago
Assignee: nobody → rrosario

Comment 4

8 years ago
(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.
(Assignee)

Comment 5

8 years ago
(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.
(Assignee)

Comment 6

8 years ago
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.
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: 2011Q3 → 2011-08-02

Comment 7

8 years ago
(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?
(Assignee)

Comment 8

8 years ago
(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

8 years ago
Created attachment 548514 [details]
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
Let's file (a) follow-up bug(s) for those.
Verified new header looks like the original attachment, the nav links match on the page & in drop down
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.