Closed
Bug 1021271
Opened 9 years ago
Closed 9 years ago
Status Bar needs refinement for 1.5x Scale and New Homescreen
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(blocking-b2g:2.0+, feature-b2g:-, b2g-v2.0 fixed, b2g-v2.1 fixed)
People
(Reporter: padamczyk, Assigned: pivanov)
References
Details
(Whiteboard: [systemsfe])
Attachments
(5 files)
The visual design on the status bar has regressed in v.2. Icons don't work anymore and the layout needs some love. Issues: 1. Notifications should be white, the blue icon clashes with the translucent status bar. 2. The time and notifications count font are too thick 3. The icons and fonts need to be horizontally aligned better, currently they are floating anchored to nothing, no baseline, no central horizon. 4. The "off" states of icons need to be more translucent. ie. When wifi is in its searching animation the "off" bars are very dark, which makes them look far too strong. Not included in this bug, but blocking this bug: * Notifications being cut off: https://bugzilla.mozilla.org/show_bug.cgi?id=1019216 * Time & AM/PM having a huge gap: https://bugzilla.mozilla.org/show_bug.cgi?id=1019200
Reporter | ||
Comment 1•9 years ago
|
||
BTW Fira font bug: https://bugzilla.mozilla.org/show_bug.cgi?id=987872
Updated•9 years ago
|
Component: Gaia::Homescreen → Gaia::System
Updated•9 years ago
|
QA Whiteboard: [VH-FL-blocking-]
Updated•9 years ago
|
QA Whiteboard: [VH-FL-blocking-] → [VH-FL-blocking-][VH-FC-blocking-]
Reporter | ||
Comment 2•9 years ago
|
||
With the reduced underlay certain colours (red and blue) clash with wallpapers. The entire set of status bar icons need to be reworked: + reduce the amount colours - get rid of the blue, make it white, can we remove more colours, like the charging yellow? + need a better solution for errors, the bright red doesn't work anymore + the dark grey / black looks terrible in the "off" states, we need to add more transparency to those forms.
Reporter | ||
Comment 3•9 years ago
|
||
Reporter | ||
Comment 4•9 years ago
|
||
Can you please provide an ETA on this fix? Thanks!
Flags: needinfo?(epang)
Reporter | ||
Updated•9 years ago
|
Comment 5•9 years ago
|
||
(In reply to Patryk Adamczyk [:patryk] UX from comment #4) > Can you please provide an ETA on this fix? Thanks! Hi Patryk, I'll look into this with Pavel today (and tomorrow if needed). We should have a fix no later then by end of this week. Thanks! Eric
Flags: needinfo?(epang)
Comment 6•9 years ago
|
||
If we're able to get these icons in, let's request approval. Moving to block bug 1017954 for now.
Comment 7•9 years ago
|
||
Hey Pavel, I've updated the sprites and put them on box here: https://mozilla.box.com/s/jcx4vwaanew1q3kk7nap Note: We talked about this before but 2.25x (madai) is different because there are only 4 signal bars instead of 5. I've left 5 for all other sizes Also, the existing sprite for 2x was extremely outdated - so there's a change that more css will need to be updated for it. I've also created a spec you can follow: https://mozilla.box.com/s/bq6hip8zqmuz0yqjc9l4 The spec covers alignment, spacing and font style/sizes. Note I also updated the gap between the time and am/pm from the other bug we already fixed (I felt like the gap was still too big after looking at it again). I believe the only assets left are the aPNGs. I'm trying to figure out how to create them, so I'll add a link to this bug when they are ready :). Thanks Pavel, if you have any questions about what I've attached so far let me know!
Assignee: epang → pivanov
Flags: needinfo?(pivanov)
Assignee | ||
Comment 8•9 years ago
|
||
Hey Eric, can you test this patch ... I'm waiting my device ... and I'm not sure how it looks like on real device
Attachment #8439878 -
Flags: ui-review?(epang)
Flags: needinfo?(pivanov)
Comment 9•9 years ago
|
||
Hi Stephany, I think this bug should be added to the 2.0 blocking list. It will update colours, alignment, text, etc. of the status bar. Let me know if you agree, thanks!
Flags: needinfo?(swilkes)
Comment 10•9 years ago
|
||
(In reply to Eric Pang [:epang] from comment #9) > Hi Stephany, I think this bug should be added to the 2.0 blocking list. It > will update colours, alignment, text, etc. of the status bar. Let me know > if you agree, thanks! Steph, just realized that this is already blocking vertical homescreen. Thanks!
Updated•9 years ago
|
Flags: needinfo?(swilkes)
Updated•9 years ago
|
blocking-b2g: --- → 2.0?
feature-b2g: --- → -
Updated•9 years ago
|
blocking-b2g: 2.0? → 2.0+
Whiteboard: [systemsfe]
Updated•9 years ago
|
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking-] → [VH-FL-blocking-][VH-FC-blocking+]
Updated•9 years ago
|
Updated•9 years ago
|
Whiteboard: [systemsfe]
Updated•9 years ago
|
Whiteboard: [systemsfe]
Comment 11•9 years ago
|
||
Hey Pavel, The status bar is looking pretty good, but there's a few changes that still need to be made. I've attached a pdf that covers the remaining changes that are needed. Some of the items were updates to the sprites/apngs which I've done, so they just need to be replaced. Let me know if you have any questions :). Flag me for ui-review when ready so we can check the changes and fix anything else that might come up if needed. Thanks!
Flags: needinfo?(pivanov)
Updated•9 years ago
|
Target Milestone: --- → 2.0 S5 (4july)
Assignee | ||
Comment 13•9 years ago
|
||
Hey Eric, as we talk few hours ago ... I fixed the issues based on your email :) and I think everything works now :) PR is updated
Flags: needinfo?(pivanov)
Comment 14•9 years ago
|
||
Comment on attachment 8439878 [details] [review] patch for Gaia/master Lots of work and back and forth here! Great work Pavel :)! R+
Attachment #8439878 -
Flags: ui-review?(epang) → ui-review+
Assignee | ||
Updated•9 years ago
|
Attachment #8439878 -
Flags: review?(21)
Comment 15•9 years ago
|
||
Comment on attachment 8439878 [details] [review] patch for Gaia/master r+ if you remove the change in apps/settings, which seems completely unrelated to this patch :)
Attachment #8439878 -
Flags: review?(21) → review+
Assignee | ||
Comment 16•9 years ago
|
||
Thanks :) Landed to master: https://github.com/mozilla-b2g/gaia/commit/e0ce5d095b767af9b371d67e0f5d1bcc1a680099
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 17•9 years ago
|
||
v2.0: https://github.com/mozilla-b2g/gaia/commit/26bd8c07f661ff01e5995991e0158ac3326d2d0f
status-b2g-v2.0:
--- → fixed
status-b2g-v2.1:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•