793790 - Use consistent auxillary nav for devhub

Status: VERIFIED FIXED

(Marketplace Graveyard :: Developer Pages, defect, P5)

Description

[Christopher Van Wiemeersch [:cvan]]

On https://marketplace-dev.allizom.org/developers/ looks like
http://f.cl.ly/items/230K08113o1m1K371G2g/Screen%20shot%202012-09-24%20at%2011.16.38%20AM.png

On https://marketplace-dev.allizom.org/developers/submissions looks like
http://f.cl.ly/items/0N1y2m3y3I1j1S1B2P2U/Screen%20shot%202012-09-24%20at%2011.15.58%20AM.png

Comment 1

[Jen Fong-Adwent [:ednapiranha]]

Chris, do you want the amo logout to be consistent and moved to the user dropdown? I took it out because you didn't want amo changed.

Also I don't think for our devhub pages 'back to apps' should be at the top b/c it's just too cluttered. I think it would be better if we keep that as limited as possible (e.g. user dropdown / log in or additionally the tools dropdown for admins).

Comment 2

[Christopher Van Wiemeersch [:cvan]]

(In reply to Jen Fong-Adwent [:ednapiranha] from comment #1)
> Chris, do you want the amo logout to be consistent and moved to the user
> dropdown? I took it out because you didn't want amo changed.

Fork them, that's fine.

> Also I don't think for our devhub pages 'back to apps' should be at the top
> b/c it's just too cluttered. I think it would be better if we keep that as
> limited as possible (e.g. user dropdown / log in or additionally the tools
> dropdown for admins).

If you can copy the auxnav used on /developers/ for /developers/submissions/ that would work well IMO.

Comment 3

[Jen Fong-Adwent [:ednapiranha]]

The headers will now be consolidated with the following fix:

- the 'logout' link will be within the user actions dropdown and the final result is one of the two:

1. non-admin: user dropdown, back to apps
2. admin: user dropdown, tools, back to apps

Comment 4

[Jen Fong-Adwent [:ednapiranha]]

merged in https://github.com/mozilla/zamboni/commit/bd17bf21cad24d83228a510adc4b7848c5e78044

Comment 5

[Jen Fong-Adwent [:ednapiranha]]

this also affects amo headers, but cvan r+ it

Comment 6

[Cristian Boldan]

Verified as fixed in https://marketplace-dev.allizom.org/developers/submissions and https://marketplace-dev.allizom.org/developers/ on FF19 (Win 7)
Postfix screencast http://screencast.com/t/tUirAsH8EC1
Closing bug.