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)
Tracking
(Not tracked)
VERIFIED
FIXED
2012-08-30
People
(Reporter: krupa.mozbugs, Assigned: spasovski)
References
()
Details
(Whiteboard: [janus])
Attachments
(2 files)
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•12 years ago
|
QA Contact: thepotch
Target Milestone: --- → 2012-08-23
Updated•12 years ago
|
Assignee: nobody → dspasovski
Comment 1•12 years ago
|
||
Also, compare the Reviews section on desktop (production) to that on mobile (-dev): http://f.cl.ly/items/2G2e1X0l0T0G3h081h3x/reviews.jpg
Updated•12 years ago
|
Priority: -- → P4
Assignee | ||
Updated•12 years ago
|
Target Milestone: 2012-08-23 → 2012-08-30
Assignee | ||
Comment 2•12 years ago
|
||
done: https://github.com/mozilla/zamboni/commit/9958196
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 3•12 years ago
|
||
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•12 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•12 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•