Rating block on details page needs some TLC

VERIFIED FIXED in 2012-08-30

Status

Marketplace
Consumer Pages
P4
normal
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: krupa, Assigned: spasovski)

Tracking

2012-08-30
Points:
---

Details

(Whiteboard: [janus], URL)

Attachments

(2 attachments)

(Reporter)

Description

6 years ago
Created attachment 652193 [details]
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.

Updated

6 years ago
QA Contact: thepotch
Target Milestone: --- → 2012-08-23

Updated

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

Updated

6 years ago
Target Milestone: 2012-08-23 → 2012-08-30
(Assignee)

Comment 2

6 years ago
done: https://github.com/mozilla/zamboni/commit/9958196
Status: NEW → RESOLVED
Last Resolved: 6 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!

Comment 4

6 years ago
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

Comment 5

6 years ago
Created attachment 658413 [details]
postfix reviews
You need to log in before you can comment on or make changes to this bug.