Closed Bug 783069 Opened 12 years ago Closed 12 years ago

Rating block on details page needs some TLC

Categories

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

defect

Tracking

(Not tracked)

VERIFIED FIXED
2012-08-30

People

(Reporter: krupa.mozbugs, Assigned: spasovski)

References

()

Details

(Whiteboard: [janus])

Attachments

(2 files)

Attached image screenshot
steps to reproduce:
1. Load https://marketplace.allizom.org/app/galactians2/?src= on your b2g phone
2. Scroll down to the reviews section

Rating block on details page needs some TLC-
1. When there are no reviews:
a) There is too much padding above the section heading
b) The --> next to 'Submit a review' link wraps to the next line.

2. When there are reviews:

a) As per the spec, we show the top two reviews (content and rating) in the details page. This is missing.
b) Below the listed reviews, there is button(?) which shows the total count and average rating for that app. Again, this is missing in the implementation.
c) Too much padding above the section heading.
QA Contact: thepotch
Target Milestone: --- → 2012-08-23
Assignee: nobody → dspasovski
Also, compare the Reviews section on desktop (production) to that on mobile (-dev):

http://f.cl.ly/items/2G2e1X0l0T0G3h081h3x/reviews.jpg
Priority: -- → P4
Target Milestone: 2012-08-23 → 2012-08-30
done: https://github.com/mozilla/zamboni/commit/9958196
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
The bylines (author names) aren't bolded like in the mocks - http://f.cl.ly/items/1Z1X3i1q2c0I1N081c3Z/Screen%20shot%202012-08-28%20at%208.58.39%20PM.png - but yeah looks totes better!
Verified as fixed in https://marketplace-dev.allizom.org/ on FF18 (Win 7 and Android 4.0.4)
Attaching postfix screenshot.
Closing bug.
Status: RESOLVED → VERIFIED
Attached image postfix reviews
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: