Closed Bug 1135142 Opened 5 years ago Closed 5 years ago
"Content Rating" should belong under the "More Information" heading
85.46 KB, image/png
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.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
This could be done pretty quickly.
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Whiteboard: [good first bug][contribute]
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.
make it so! font-weight 500!
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.
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: 5 years ago → 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2015-03-24
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
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.