1135142 - "Content Rating" should belong under the "More Information" heading

Status: VERIFIED FIXED

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

Description

[Christopher Van Wiemeersch [:cvan]]

Attachment: screenshot of "MORE INFORMATION" and "CONTENT RATING"

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.

Comment 1

[Wil Clouser [:clouserw]]

This could be done pretty quickly.

Comment 2

[Christopher Van Wiemeersch [:cvan]]

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

[David Durst [:ddurst]]

make it so! font-weight 500!

Comment 4

[Philip [:pwalm]]

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.

Comment 5

[Kevin Ngo [:ngoke]]

I made it look like part of the def list on mobile:

https://github.com/mozilla/fireplace/commit/6deb7ddb782d4c643a28f1560fbd91facb03d036

Comment 6

[Madalin Cotetiu]

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.