Closed
Bug 1135142
Opened 8 years ago
Closed 8 years ago
"Content Rating" should belong under the "More Information" heading
Categories
(Marketplace Graveyard :: Consumer Pages, defect, P4)
Tracking
(Not tracked)
VERIFIED
FIXED
2015-03-24
People
(Reporter: cvan, Assigned: kngo)
References
()
Details
(Whiteboard: [good first bug][contribute][ktlo])
Attachments
(1 file)
85.46 KB,
image/png
|
Details |
On desktop or mobile, load an app detail page <https://marketplace.firefox.com/app/zirma>. It's odd to see the "CONTENT RATING" heading be capitalised below the "Most Recent Version" heading. Then I noticed the "MORE INFORMATION" up top. I would do 1. Make the "CONTENT RATING" h3 become part of the definition list (i.e., not uppercased). 2. Potentially, remove the "MORE INFORMATION" heading completely.
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Comment 1•8 years ago
|
||
This could be done pretty quickly.
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Whiteboard: [good first bug][contribute]
Reporter | ||
Comment 2•8 years ago
|
||
FWIW, I want to reiterate my concern here in comment 0: > It's odd to see the "CONTENT RATING" heading be capitalised below the "Most Recent Version" heading. It looks like a bug/oversight that there headings of the same font-size but one is capitalised and the other is not. It's not enough differentiation, and it just looks wrong.
Comment 3•8 years ago
|
||
make it so! font-weight 500!
Updated•8 years ago
|
Severity: normal → minor
Priority: -- → P4
Updated•8 years ago
|
Flags: needinfo?(pwalmsley)
Comment 4•8 years ago
|
||
On desktop this works, because they're in two different columns. But yeah, it does looks a bit off on mobile. My suggestion would be to put "Content Rating" into its own white block.
Flags: needinfo?(pwalmsley)
Assignee | ||
Comment 5•8 years ago
|
||
I made it look like part of the def list on mobile: https://github.com/mozilla/fireplace/commit/6deb7ddb782d4c643a28f1560fbd91facb03d036
Assignee: nobody → kngo
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2015-03-24
Comment 6•8 years ago
|
||
Verified as fixed in FF39(Win7) on mobile width FFOS 2.0(Flame), FFOS 1.4(Inari) and Android 4.2.1 Desktop: http://screencast.com/t/vJJjk9yuf Android: http://screencast.com/t/Vdno3VUh7 FFOS: http://screencast.com/t/2ZCJKnyr Closing bug.
Status: RESOLVED → VERIFIED
Updated•8 years ago
|
Whiteboard: [good first bug][contribute] → [good first bug][contribute][ktlo]
You need to log in
before you can comment on or make changes to this bug.
Description
•