Closed Bug 1069525 Opened 7 years ago Closed 7 years ago
(Settings) Network > Data Dialogs are using the wrong styles
The fonts in this dialog are really small, most of our dialogs use bigger text. This affects "Data Connection" and "Data Roaming" dialogs.
Hey Patryk, we use 2.2rem font size for paragraphs and 1.6rem for titles. I think if there are bigger fonts in some dialogs they don't use [BB]. Do we need to change something on the current dialogs?
Przemek do we have dialog spec ready, it would be good to get Pavel to work from that.
Flags: needinfo?(padamczyk) → needinfo?(pabratowski)
Ignore comment 2. Pavel, I think the issue here is that the dialog fonts are too small, they should be bigger. Ie look at the dialog when you try to access Music and have the phone plugged in via USB. The sizing is way different.
Hey Patryk, here are few shots.
Attachment #8501608 - Flags: ui-review?(padamczyk)
Attachment #8500992 - Flags: ui-review?(padamczyk) → ui-review+
Comment on attachment 8501608 [details] Device Shots Looks good, ship it!
Attachment #8501608 - Flags: ui-review?(padamczyk) → ui-review+
Attachment #8500992 - Flags: review?(ehung)
Attachment #8500992 - Flags: review?(ehung) → review?(arthur.chen)
Comment on attachment 8500992 [details] [review] patch for Gaia/master r=me, thanks!
Attachment #8500992 - Flags: review?(arthur.chen) → review+
Thanks :) Landed to master: https://github.com/mozilla-b2g/gaia/commit/591c2c5cda69232d88c30e8b5e8a44602bd0a54f
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
This is an important message around user cost and thus the sense of security. It must have high readability. Please fix
blocking-b2g: --- → 2.1?
Blocking- for not meeting blocker criteria: https://wiki.mozilla.org/B2G/Triage#Blocker_Triage_Guidelines However, request approval on the patch to get an uplift to 2.1.
blocking-b2g: 2.1? → -
hi Dietrich, may i know if you have requested approval to land this patch to 2.1?
(In reply to Francis Lee [:frlee] from comment #11) > hi Dietrich, > > may i know if you have requested approval to land this patch to 2.1? Nope. I was recommending that whoever wants it landed should do so.
You need to log in before you can comment on or make changes to this bug.