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

VERIFIED FIXED in 2015-03-24

Status

defect
P4
minor
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: cvan, Assigned: kngo)

Tracking

Avenir
2015-03-24
Dependency tree / graph

Details

(Whiteboard: [good first bug][contribute][ktlo], )

Attachments

(1 attachment)

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: 4 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!
Severity: normal → minor
Priority: -- → P4
Blocks: 1135257
Flags: needinfo?(pwalmsley)

Comment 4

4 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)
No longer blocks: 1135257
Blocks: 1135257
Assignee

Comment 5

4 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: 4 years ago4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2015-03-24

Comment 6

4 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
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.