755922 - [Ratings] Show ratings on detail page

Status: VERIFIED FIXED

(Marketplace Graveyard :: Consumer Pages, defect, P1)

Description

[Christopher Van Wiemeersch [:cvan]]

Desktop:
* Left column
    - Show the five most recent positive ratings
    - Heading should have a thumbs-up icon and indicate the total count
    - Link to view all ratings
* Right column
    - Show the five most recent negative ratings
    - Heading should have a thumbs-down icon and indicate the total count
    - Link to view all ratings

Mobile:
* Collapse the columns so the positive ratings are first - followed by the negative ones

Comment 1

[Christopher Van Wiemeersch [:cvan]]

The fields we want for the widget of each individual rating/review:

* by {Author}
* Timestamp
* Permalink
* Thumbs Up / Thumbs Down
* Comment
* Report/Flag as Spam
* Delete (Consumer, Admin, Reviewer)
    - https://github.com/mozilla/zamboni/blob/master/apps/reviews/helpers.py#L62

Comment 2

[Christopher Van Wiemeersch [:cvan]]

Be sure to show reviews *only* if the app is public!

Comment 3

[Christopher Van Wiemeersch [:cvan]]

WIP:

https://github.com/mozilla/zamboni/commit/4d7e053
https://github.com/mozilla/zamboni/commit/8666392

Comment 4

[Christopher Van Wiemeersch [:cvan]]

https://github.com/mozilla/zamboni/commit/d748e6d

Comment 5

[krupa raj[:krupa]]

How does this work for desktop+tablet? Should we show the latest n of reviews on the details page?

Comment 6

[Christopher Van Wiemeersch [:cvan]]

Desktop: http://cl.ly/2z0K2a0s1h3H1l1G2v1m
Tablet: http://cl.ly/2d2o371x1L2K2D0y0Q20
Mobile: http://cl.ly/2E3v1o0o2U1j2O19263M

I think there's enough info on Mobile. A button to Reviews is sufficient. How should this look on Desktop and Tablet?

Comment 7

[Maria Sandberg [:mushi]]

This should now be covered by bug 761906 comment 10

Comment 8

[Christopher Van Wiemeersch [:cvan]]

(In reply to msandberg from comment #7)
> This should now be covered by bug 761906 comment 10

This covers the mobile version. But we're not sure how this would look on Desktop.

Comment 9

[Christopher Van Wiemeersch [:cvan]]

Kevin to work with Maria on figuring out how this looks on desktop.

Comment 10

[Christopher Van Wiemeersch [:cvan]]

(In reply to Chris Van Wiemeersch [:cvan] from comment #9)
> Kevin to work with Maria on figuring out how this looks on desktop.

Maria, how should this look on desktop?

Comment 11

[Maria Sandberg [:mushi]]

(In reply to Chris Van Wiemeersch [:cvan] from comment #10)
> > Kevin to work with Maria on figuring out how this looks on desktop.
> Maria, how should this look on desktop?

Desktop designs are not quite done yet. Expect to see them here next week.

Comment 12

[Christopher Van Wiemeersch [:cvan]]

This should do until we get real desktop designs:

http://f.cl.ly/items/3v0z1S1H0h3f3R450A0O/Reviews-small.jpg

Comment 13

[Kevin Ngo [:ngoke]]

https://github.com/mozilla/zamboni/commit/883f1aa926d3f69bf26e8c5eae220c6fbf3bcfbb

Comment 14

[Christopher Van Wiemeersch [:cvan]]

A few visual nitpicks:

http://f.cl.ly/items/3m0n2B462E1v031V381X/Screen%20shot%202012-07-30%20at%204.21.52%20AM.png
http://f.cl.ly/items/1G420H0d11131h1f0Q2M/Screen%20shot%202012-07-30%20at%204.24.42%20AM.png

Comment 15

[Christopher Van Wiemeersch [:cvan]]

Also, the "Reviews" heading should be the same size as that of the other support headings: http://f.cl.ly/items/0Q2h251r1D1X0x2H141s/Screen%20shot%202012-07-30%20at%2010.58.07%20AM.png

Comment 16

[Christopher Van Wiemeersch [:cvan]]

https://github.com/mozilla/zamboni/commit/78fc209
https://github.com/mozilla/zamboni/commit/a464f33
https://github.com/mozilla/zamboni/commit/aab3ff4

Comment 17

[Cristian Boldan]

Verified as fixed in https://marketplace-dev.allizom.org/ on FF20 (Win 7)
Postfix screencast http://screencast.com/t/KpWOPBrSr8mN
Closing bug.