[Ratings] Show ratings on detail page

VERIFIED FIXED in 2012-08-02

Status

P1
normal
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: cvan, Assigned: kngo)

Tracking

({uiwanted})

2012-08-02
uiwanted
Points:
---
Dependency tree / graph

Details

(Reporter)

Description

6 years ago
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
(Reporter)

Comment 1

6 years ago
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
(Reporter)

Updated

6 years ago
Depends on: 755925
(Reporter)

Updated

6 years ago
Depends on: 755927
(Reporter)

Updated

6 years ago
Depends on: 755931
(Reporter)

Updated

6 years ago
No longer depends on: 755925
(Reporter)

Updated

6 years ago
No longer depends on: 755927
(Reporter)

Updated

6 years ago
No longer depends on: 755931
(Reporter)

Comment 2

6 years ago
Be sure to show reviews *only* if the app is public!
(Reporter)

Updated

6 years ago
Blocks: 757227
Target Milestone: 2012-05-17 → 2012-06-07
(Reporter)

Comment 4

6 years ago
https://github.com/mozilla/zamboni/commit/d748e6d
Assignee: thepotch → cvan
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Reporter)

Updated

6 years ago
Target Milestone: 2012-06-07 → 2012-06-14

Comment 5

6 years ago
How does this work for desktop+tablet? Should we show the latest n of reviews on the details page?
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Reporter)

Comment 6

6 years ago
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?
(Reporter)

Updated

6 years ago
Depends on: 764670
(Reporter)

Updated

6 years ago
Target Milestone: 2012-06-14 → 2012-06-21
(Reporter)

Updated

6 years ago
Whiteboard: [ui]
Keywords: uiwanted
Whiteboard: [ui]
This should now be covered by bug 761906 comment 10
(Reporter)

Comment 8

6 years ago
(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.
(Reporter)

Comment 9

6 years ago
Kevin to work with Maria on figuring out how this looks on desktop.
Assignee: cvan → ngoke
Target Milestone: 2012-06-21 → 2012-07-12
(Assignee)

Updated

6 years ago
Target Milestone: 2012-07-12 → 2012-07-26
(Reporter)

Comment 10

6 years ago
(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?
(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.
(Assignee)

Updated

6 years ago
Target Milestone: 2012-07-26 → 2012-08-02
(Reporter)

Comment 12

6 years ago
This should do until we get real desktop designs:

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

Comment 15

6 years ago
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
(Reporter)

Comment 16

6 years ago
https://github.com/mozilla/zamboni/commit/78fc209
https://github.com/mozilla/zamboni/commit/a464f33
https://github.com/mozilla/zamboni/commit/aab3ff4
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago6 years ago
Resolution: --- → FIXED

Comment 17

6 years ago
Verified as fixed in https://marketplace-dev.allizom.org/ on FF20 (Win 7)
Postfix screencast http://screencast.com/t/KpWOPBrSr8mN
Closing bug.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.