Closed
Bug 848961
Opened 8 years ago
Closed 8 years ago
Restrict to 4 font sizes in the UI
Categories
(Firefox for Android Graveyard :: Theme and Visual Design, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 22
People
(Reporter: sriram, Assigned: sriram)
Details
Attachments
(1 file)
15.87 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
We currently have 8,10,12,13,14,15,16,18 font-sizes in our app. It's better use: Micro - 12sp Small - 14sp Normal - 16sp Medium - 18sp Large - 22sp in our app.
Assignee | ||
Comment 1•8 years ago
|
||
First step in getting everything in orderly manner. (and no. i'm not talking about string theory ;) )
Attachment #722464 -
Flags: review?(mark.finkle)
Comment 2•8 years ago
|
||
Comment on attachment 722464 [details] [diff] [review] Patch I think screenshots are needed to test/verify on various phone and tablet screensizes.
Attachment #722464 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 3•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/02d63d7e6c87
Comment 4•8 years ago
|
||
Backed out the whole push in https://hg.mozilla.org/integration/mozilla-inbound/rev/5eb7efc431c8 for mochitest-1 and robocop-2 errors about "Error inflating class android.widget.Button" like https://tbpl.mozilla.org/php/getParsedLog.php?id=20799813&tree=Mozilla-Inbound and https://tbpl.mozilla.org/php/getParsedLog.php?id=20798777&tree=Mozilla-Inbound
Comment 5•8 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/02d63d7e6c87
Assignee: nobody → sriram
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 22
Assignee | ||
Comment 6•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/b225cfbfda52
Updated•2 months ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•