519865 - Add Personas detail page

Status: VERIFIED FIXED

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

Description

[Wil Clouser [:clouserw]]

This is 1.7 in the spec:  The persona listing pages will be hosted on AMO and contain the same data as the 'details' page on getpersonas.com.  The actual personas and images will be served off of getpersonas.com.  If a user doesn't have the personas add-on installed, it will be installed when the user clicks 'get persona'.  User names will be linked to user profiles on getpersonas.com.  There should also be 'learn more about personas' links that point to getpersonas.com.  This is similar in principle to non-hosted add-ons on AMO (which we've been wanting to do for a while).

rdoherty is rewriting the download buttons in this release so please coordinate with him since this will affect those.

Comment 1

[Jeff Balogh (:jbalogh)]

Are we going to do the hover trick on AMO?  Is there anything special I need to know to do that?

Comment 2

[Wil Clouser [:clouserw]]

It's JS as far as I know.  We should be able to steal it from getpersonas.c

Comment 3

[Ryan Doherty (:rdoherty)]

There's a JSON blob on the element that contains the persona info. Any page with a persona image will have it there. 

persona="{"id":"33","name":"Groovy Blue","accentcolor":"#6699ff","textcolor":"#07188d","header":"3\/3\/33\/tbox-groovy_blue.jpg","footer":"3\/3\/33\/stbar-groovy_blue.jpg","category":"Abstract","description":null,"author":"Lee.Tom","detailURL":"http:\/\/www.getpersonas.com\/persona\/33","headerURL":"http:\/\/www.getpersonas.com\/static\/3\/3\/33\/tbox-groovy_blue.jpg","footerURL":"http:\/\/www.getpersonas.com\/static\/3\/3\/33\/stbar-groovy_blue.jpg","previewURL":"http:\/\/www.getpersonas.com\/static\/3\/3\/33\/preview.jpg","iconURL":"http:\/\/www.getpersonas.com\/static\/3\/3\/33\/preview_small.jpg"}"

JS is here, which contains personas add-on detection and events for previewing/loading personas.

http://viewvc.svn.mozilla.org/vc/projects/getpersonas.com/trunk/static/js/script.js?revision=52457&view=markup

JS fires events on that element which the Personas add-on or 3.6 catches and loads the image. 

We will be moving images to a CDN so be prepared for the image urls to change.

Comment 4

[Jeff Balogh (:jbalogh)]

Attachment: personas detail view

Comment 5

[Jeff Balogh (:jbalogh)]

thanks, r53277

Comment 6

[Jeff Balogh (:jbalogh)]

QA: Note that if you don't have preview.amo in your list of security-excepted sites, you won't be able to preview or install personas.

Comment 7

[Vishal Kamdar [:vish_moz]]

According to the description,  https://bugzilla.mozilla.org/show_bug.cgi?id=519865#c0 there should be a 'learn more about personas' link. This link is missing.

Comment 8

[Vishal Kamdar [:vish_moz]]

also filed Bug 522407 related to personas detail page.

Comment 9

[Jeff Balogh (:jbalogh)]

Added "What are Personas?" in r53475.  Flig/nick/anyone, what's a better place for the link than it's current spot?  I just dropped it in so Wil could push 110n.

Comment 10

[Jeff Balogh (:jbalogh)]

styled in r53595.

Comment 11

[Nick Nguyen [:osunick]]

I think the link is fine where it is- let's just leave it there!

Comment 12

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

Verified FIXED on https://preview.addons.mozilla.org/en-US/firefox/persona/15232.