848961 - Restrict to 4 font sizes in the UI

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Theme and Visual Design, defect)

Description

[Sriram Ramasubramanian [:sriram]]

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.

Comment 1

[Sriram Ramasubramanian [:sriram]]

Attachment: Patch

First step in getting everything in orderly manner. (and no. i'm not talking about string theory ;) )

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 722464 [details] [diff] [review]
Patch

I think screenshots are needed to test/verify on various phone and tablet screensizes.

Comment 3

[Sriram Ramasubramanian [:sriram]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/02d63d7e6c87

Comment 4

[Phil Ringnalda (:philor)]

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

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/02d63d7e6c87

Comment 6

[Sriram Ramasubramanian [:sriram]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b225cfbfda52

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/b225cfbfda52