Closed Bug 1042713 Opened 7 years ago Closed 7 years ago

[User Story] Remove notification icons from status bar


(Firefox OS Graveyard :: Gaia::System, defect)

Not set


(tracking-b2g:backlog, b2g-v2.1 fixed)

2.1 S3 (29aug)
tracking-b2g backlog
Tracking Status
b2g-v2.1 --- fixed


(Reporter: pdol, Assigned: apastor)



(Keywords: feature, Whiteboard: [ucid:System246], [ft:systemsfe])

User Story

As a user, I want my notification icons removed from the status bar so as to make more room for other status bar icons.

Acceptance Criteria:
1. The old notification indicator is no longer displayed in the status bar.
2. When I have new notifications that I have not looked at, a new indicator is used to inform me of them, per UX spec.


(5 files, 1 obsolete file)

No description provided.
Alberto, can you help out with this?

Note: for the ambient notification toaster animation, I have a demo from a while ago that has some keyframes that might give you a jump start here:
Assignee: nobody → apastor
Update Spec
Attachment #8479388 - Attachment is obsolete: true
Sorry for the delay.
Attaching the first approach for the ambient notification implementation. Probably still some tweaks in terms of timings of the animation to be done, but I'll like to start getting some feedback.

Attachment #8480709 - Flags: feedback?(mhenretty)
Attachment #8480709 - Flags: ui-review?(epang)
Duplicate of this bug: 1052222
Attachment #8480709 - Flags: ui-review?(rmacdonald)
Comment on attachment 8480709 [details] [review]
Ambient notifications first approach

Plus'ing based on the IxD / behaviour but this is subject to Eric's visual review. I'll also email Eric about some potential visual polish and ways to improve contrast in some scenarios.
Attachment #8480709 - Flags: ui-review?(rmacdonald) → ui-review+
Just realized one issue. The portion of the ambient indicator that varies based on the number of notifications should be based on the number of <em>unseen</em> notifications as opposed to the number of total notifications. I'll double check the spec and make any changes.
Comment on attachment 8480709 [details] [review]
Ambient notifications first approach

Hey Alberto,

Thanks for working on this!  It's look really good, but there are some visual changes that are needed.

Notification Toast:

1. The Notification Icon should be vertically centered in the toast

2. Calls, SMS (and others that are two lines) should include both lines.  For example, for SMS, The Name should be on the first line (18px, regular) and the message on the second (14px, regular).

3.  For single line notifications - They should be vertically centered with the notification icon.

4.  The background colour of the toast should be #00BCE2 (this will make sense with #3 below)

Indicator Bar:

1. Can we reduce the indicator by 1px?  It's currently to spec, but seeing it implemented Rob is right, it looks to dominant as is. 

2. The handle should transition from white to #B3F3FF (while the opacity increases) 

3. When the handle expands to the sides of the screen (for the first notification) it should transition from #B3F3FF to #00BCE2

Thanks again!  Please let me know if you have any questions!
Attachment #8480709 - Flags: ui-review?(epang) → ui-review-
Flags: needinfo?(alberto.pastor)
Hi Eric,

Thanks for the feedback!
Regarding the notification toast, I didn't touch it at all. I thought this bug was only about removing the notifications indicator in the status bar (and replace it with the ambient indicator).

Should I fix the toast as part of this bug as well?

Regarding the indicator, I'll fix it today.

Flags: needinfo?(alberto.pastor)
Comment on attachment 8480709 [details] [review]
Ambient notifications first approach

Thanks for making all the changes!  It looks great now :) R+
Attachment #8480709 - Flags: ui-review- → ui-review+
Comment on attachment 8480709 [details] [review]
Ambient notifications first approach

LGTM! Probably ready for the real r? after tests are fixed :)
Attachment #8480709 - Flags: feedback?(mhenretty) → feedback+
Attachment #8480709 - Flags: review?(mhenretty)
Comment on attachment 8480709 [details] [review]
Ambient notifications first approach

A couple of nits, but overall this is very cool to see!
Attachment #8480709 - Flags: review?(mhenretty) → review+
Closed: 7 years ago
Resolution: --- → FIXED
feature-b2g: --- → 2.1
Reverted in master for causing bug 1060627.

We are going to squash the fix from bug 1060627 into this when we reland.
Resolution: FIXED → ---
No longer depends on: 1060627
Duplicate of this bug: 1060627
This is Alberto's patch with the fix from bug 1060627 squashed into his commit. Waiting for green before relanding.
Attachment #8481633 - Flags: review+
feature-b2g: 2.1 → ---
Closed: 7 years ago7 years ago
Resolution: --- → FIXED
Depends on: 1060935
Duplicate of this bug: 1044115
Duplicate of this bug: 1054312
Duplicate of this bug: 1047251
Target Milestone: --- → 2.1 S3 (29aug)
No longer blocks: 1063601
Blocks: 1063601
blocking-b2g: backlog → ---
You need to log in before you can comment on or make changes to this bug.