512773 - [1.1] Developer Hub Homepage

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Developer Pages, defect, P1)

Description

[Justin Scott [:fligtar]]

Attachment: mockup

Please see section 1.1 of the spec linked above for details.

This bug is for implementation of the homepage at addons.mozilla.org/developers, in accordance with the spec and mock-up attached with the following details:

* The title of the site will be Add-on Developer Hub, although that should be taken care of in the global header/nav bug.
* The promo box (large box with the fox) changes based on whether the user is logged into AMO with at least one add-on submitted. This bug is for implementing the logged-out/no-addons state, which has the "Are you ready to make browsing better?" text.
* The promo box copy is not ready yet. Please include some placeholder text that I will update later.
* The Start Making Add-ons button should link to the Getting Started page.
* The Ready to Share? box's button should link to the developer tools add-on submission form.
* The Developer Blog section should be called Developer News and pull the 5 latest blog entries from http://blog.mozilla.com/addons/category/developers/feed/
* The Upcoming Events box should be created, but I will file a separate bug for implementation of the dynamic events.
* The links at the bottom have changed from the mock-up placeholders. They will be:
-> Learn & Explore
** How-to Library
** API & Language Reference
** Case Studies
-> Create & Connect
** Developer Forums
** Add-on Builder
** Add-on Validator
The copy for under these links will be forthcoming. Please use placeholders for now.

I think it will probably be easiest for l10n if we make strings localizable but don't add them to the .pos until the final copy is in place.

Comment 1

[Justin Scott [:fligtar]]

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.

Comment 2

[Ryan Doherty (:rdoherty)]

Can I get the psds/ai files for this?

Comment 3

[Fred Wenzel [:wenzel]]

My guess is: http://people.mozilla.org/~chowse/drop/developer_amo/masters/

Comment 4

[Ryan Doherty (:rdoherty)]

Attachment: check-in

partial patch for quick review to make sure I'm not doing anything terribly wrong.

Comment 5

[Ryan Doherty (:rdoherty)]

Attachment: v1

Full patch. 

Notes:
* The main promo module at the top shrinks as the browser window narrows. Tried to get it to work as best I could without the bg image going under the text or the text becoming too narrow. 
* Most links have no urls yet.
* Didn't implement calendar code as that's another bug. 
* Created a migration, but no rev # yet as code hasn't been committed
* run `bin/maintenance.php blog` to fetch blog posts
* Not sure if more error checking is needed when fetching blog posts
* No detection for if the user has add-ons, will address that in bug 512778

Comment 6

[Ryan Doherty (:rdoherty)]

Attachment: v2

RTL fixes that I forgot.

Comment 7

[Wil Clouser [:clouserw]]

Comment on attachment 399805 [details] [diff] [review]
v2

Looking great.  Two things before committing:

1) Please add 'true' as a second parameter to: debug('Could not fetch blog feed');

2) in addons_html.php you have some strings like "Posted %d days ago".  Any string with a plural needs to use n___() even if it doesn't make sense in English, some locales have 5 or 6 plural forms and they'll need to differentiate them.

Comment 8

[Ryan Doherty (:rdoherty)]

really messed up my tree with git, committed v2 patch, will add fixes on monday.

r51335

Comment 9

[Justin Scott [:fligtar]]

Thanks Ryan. I added the copy and links in r51371.

A couple comments:
* In the directory links section, there's a featured class that you have on case studies that gives it a blue background. I know this in the mockup, and at the time I assumed it was an on-hover thing, but I think we can remove it completely.

* The RSS icon should link to the feed.

* I think I put this in the Upcoming Events box bug, but since you ended up implementing it, we should remove the calendar icon.

Thanks!

Comment 10

[Ryan Doherty (:rdoherty)]

r51430 fixes issues in comments 7 & 9

Comment 11

[Stephen Donner [:stephend] Not actively reading bugmail]

I went through this line by line and think I got them all covered as I was testing https://preview.addons.mozilla.org/en-US/firefox/developers.

Verified FIXED -- hub is looking great!