519859 - Add "Browse Personas" page

Status: VERIFIED FIXED

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

Description

[Wil Clouser [:clouserw]]

This is 1.2 in the spec - your favorite page, a modified themes page! Mockup available in attachment 394072 [details].  The categories should use the existing categories on http://www.getpersonas.com/en-US/gallery/All/Popular

Comment 1

[Jeff Balogh (:jbalogh)]

Attachment: personas list view

no personas onhover js, that comes later

Comment 2

[Jeff Balogh (:jbalogh)]

Attachment: personas list view

with all the goodies this time

Comment 3

[Jeff Balogh (:jbalogh)]

Attachment: personas list view

rdoherty pointed out that I have too much nesting

Comment 4

[Fred Wenzel [:wenzel]]

Comment on attachment 405535 [details] [diff] [review]
personas list view

I tried it out and it works very well. There's a huge amount of personas, wow.

Also thanks for cleaning up trailing whitespace (I try to do that too) but patches are easier to read if you exclude those lines, then sneak them in when you commit :)

Comment 5

[Jeff Balogh (:jbalogh)]

Thanks! r53162

QA: note that /persona/<id> detail pages are bug 519865, which is still open at the moment.

Comment 6

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

QA testable here: https://preview.addons.mozilla.org/en-US/firefox/personas/

Comment 7

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

Verified FIXED; we've been testing (and filing one-off bugs for this).