[1.8] Developer Hub global header and navigation bar

VERIFIED FIXED in 5.1

Status

addons.mozilla.org Graveyard
Developer Pages
P1
normal
VERIFIED FIXED
9 years ago
3 years ago

People

(Reporter: fligtar, Assigned: wenzel)

Tracking

unspecified

Details

(URL)

Attachments

(9 attachments, 2 obsolete attachments)

(Reporter)

Description

9 years ago
Created attachment 396842 [details]
mockup

Please see section 1.8 of the spec linked above and the attached mock-ups for details.

Every page of the developer hub should have the same header and nav bar. The header should say "Add-on Developer Hub". We will have a logo to be displayed in the header, but it's not ready yet. The header text should be the same position and colors as in the attached homepage mockup, not as the header is arranged in the template mockup.

The navbar menu items should be (different from mockup):

-> My Add-ons
* [list of add-ons]
* Submit New Add-on

-> Documentation
* Getting Started
* Add-on Policies
* Case Studies
* How-to Library
* API & Language Reference

-> Tools
* Add-on Builder
* Add-on Validator

-> Community
* Blog
* Forums
* Newsletter

some notes:
* My Add-ons should only appear for developers. Clicking the My Add-ons item should go to the dashboard listing all the user's add-ons.
* The list of add-ons should be in alphabetical order and go up to 7. If there are more than 7 add-ons, instead of the 7th add-on, display a "more add-ons..." link that goes to the dashboard listing all add-ons.

* The search box should say "Search add-on resources" and remove that text when clicked. Hitting enter or the green button will GET the search page.

* There will be a link to get back to "normal" AMO that I need to figure out where to place.
(Reporter)

Comment 1

9 years ago
Created attachment 396843 [details]
homepage mock for header reference
(Reporter)

Comment 2

9 years ago
Created attachment 397120 [details]
updated template mockup

This is an updated template mock-up with the very top navigation changes for the developer hub.

Namely, instead of the Other Applications menu, there should be a "Back to Add-ons" link that goes to the normal homepage.
Attachment #396842 - Attachment is obsolete: true
(Reporter)

Updated

9 years ago
Assignee: nobody → fwenzel
(Reporter)

Comment 3

9 years ago
Any assets needed for the Developer Hub should be here: http://people.mozilla.org/~chowse/drop/developer_amo/v4/assets/common/ If you find something missing or have any questions on style, please ask.
(Assignee)

Comment 4

9 years ago
Created attachment 397587 [details] [diff] [review]
JS cleanup

Like I promised the last time, you'll get these changes in more easily digestible chunks now. :)

This first patch moves add-ons display page-only crap out of the layout and into the addons display page where it belongs.
Attachment #397587 - Flags: review?(clouserw)
(Assignee)

Comment 5

9 years ago
Created attachment 397588 [details] [diff] [review]
Header element

This second patch removes the public pages header from the layout and drops it into an element, as well as the user/login part into a separate element so I can reuse it on the developer side of things.
Attachment #397588 - Flags: review?(clouserw)
(Assignee)

Comment 6

9 years ago
Created attachment 397589 [details] [diff] [review]
Initial DevHub header

This third patch adds a dev hub header as well as a skeleton for rdoherty to flesh out (haha) in bug 512773.

The only piece missing from this is the blue navigation bar, which will get another (and probably final, for this bug) patch.
Attachment #397589 - Flags: review?(clouserw)
(Assignee)

Comment 7

9 years ago
(In reply to comment #6)
> Created an attachment (id=397589) [details]
> Initial DevHub header

The header image was linking to the front page, not developer hub front page. Ignore this please, I fixed it locally already.
(Assignee)

Comment 8

9 years ago
Chowse, fligtar: I could still use both the magnifying glass and the arrow in the search box. For extra credit, the latter would also be helpful in an rtl-version.
(Reporter)

Comment 9

9 years ago
Fred, can you use the arrow/magnifying glass from the normal search bar? I know it's slightly different, but if it looks too odd we can replace it after.
(Assignee)

Comment 10

9 years ago
Will try it out like that, thanks.
(Assignee)

Comment 11

9 years ago
Created attachment 397698 [details] [diff] [review]
Nav bar

This patch styles the main navigation bar.

Fligtar: I assigned the existing URLs to the respective links in the "my add-ons" tab but all the other ones are unassigned. If you have any URLs in mind for any of them, let me know, otherwise it's no big deal and we should change the "TODO" links as the pages in question are created. We just need to make sure all of them work properly before we ship this.
Attachment #397698 - Flags: review?(clouserw)
(Assignee)

Comment 12

9 years ago
Created attachment 397699 [details]
Screenshot of the nav bar

Screenshot of the nav bar, for everyone's enjoyment.
(Assignee)

Comment 13

9 years ago
Created attachment 397702 [details] [diff] [review]
Put the dashboard in a new place

This final patch (promised) puts the former developer dashboard in a new place (developers/addons) as it's not the dashboard anymore. The old URL stays valid though for now.

This patch doesn't skin that page yet, that'll happen in bug 513150.
Attachment #397702 - Flags: review?(clouserw)
(Assignee)

Updated

9 years ago
Status: NEW → ASSIGNED
(Reporter)

Comment 14

9 years ago
Thanks for working on this so quickly, Fred!

My original comment on this was probably pretty confusing, but the header text should look like attachment 396843 [details] without the logo.

And I'll have chowse give us the assets for the search bar stuff, as I do like his version better.
(Assignee)

Comment 15

9 years ago
(In reply to comment #14)
> My original comment on this was probably pretty confusing, but the header text
> should look like attachment 396843 [details] without the logo.

But it is called "Developer Hub" still, right? Just the tools logo goes away?

> And I'll have chowse give us the assets for the search bar stuff, as I do like
> his version better.

All right.
(Assignee)

Comment 17

9 years ago
Created attachment 397829 [details]
Screenshot of the nav bar with new assets

This is what it looks like with the new assets. Wil, I won't give you a new patch for this, but I added this locally.

I will also add an rtl screenshot shortly, just because I can (albeit untranslated as both my Hebrew and Arabic are slightly rusty ;))
Attachment #397699 - Attachment is obsolete: true
(Assignee)

Comment 18

9 years ago
Created attachment 397831 [details]
rtl screenshot
Comment on attachment 397587 [details] [diff] [review]
JS cleanup

These are looking good.  They mess up the CSS on the rest of the dev panel, but I guess that's to be expected.  Thanks wenzel.
Attachment #397587 - Flags: review?(clouserw) → review+
Attachment #397588 - Flags: review?(clouserw) → review+
Attachment #397589 - Flags: review?(clouserw) → review+
Attachment #397698 - Flags: review?(clouserw) → review+
Attachment #397702 - Flags: review?(clouserw) → review+
(Assignee)

Comment 20

9 years ago
I just landed this: Pretty much all revisions between r50610 and r50621.
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Keywords: push-needed
Resolution: --- → FIXED
Verified FIXED on https://preview.addons.mozilla.org/en-US/developers.
Status: RESOLVED → VERIFIED
Keywords: push-needed
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.