If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Use consistent auxillary nav for devhub

VERIFIED FIXED

Status

Marketplace
Developer Pages
P5
normal
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: cvan, Assigned: ednapiranha)

Tracking

Points:
---

Details

(URL)

(Reporter)

Description

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

Comment 1

5 years ago
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).
(Reporter)

Comment 2

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

Updated

5 years ago
Assignee: nobody → jfong
(Assignee)

Comment 3

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

Comment 4

5 years ago
merged in https://github.com/mozilla/zamboni/commit/bd17bf21cad24d83228a510adc4b7848c5e78044
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Assignee)

Comment 5

5 years ago
this also affects amo headers, but cvan r+ it

Comment 6

5 years ago
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.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.