Closed
Bug 1073480
Opened 10 years ago
Closed 10 years ago
[Notifications] Ambient Indicator - Remove count indicator
Categories
(Firefox OS Graveyard :: Gaia::System, defect, P1)
Tracking
(tracking-b2g:backlog, b2g-v2.1 verified, b2g-v2.2 verified)
People
(Reporter: epang, Assigned: apastor)
Details
(Keywords: polish, Whiteboard: [systemsfe])
Attachments
(3 files)
(deleted),
image/jpeg
|
Details | |
(deleted),
image/jpeg
|
Details | |
(deleted),
text/x-github-pull-request
|
etienne
:
review+
epang
:
ui-review+
bajaj
:
approval-gaia-v2.1+
|
Details |
Hi Alberto,
The UX team has decided to remove the notification counter from the ambient indicator. The decision was made since a lot of people were confused by it and thought it might be broken.
I've attached an updated spec. Now when the toast goes up the bar fades to #00d3ff.
Please let me know if you have any questions, I'm hoping this can be a quick fix. Thanks!
Reporter | ||
Comment 1•10 years ago
|
||
Reporter | ||
Comment 2•10 years ago
|
||
[Blocking Requested - why for this release]:
Based on feedback many people find that the notification counter to look like a broken animation. It has also caused a lot of confusion, because of this UX and Visual have decided to simplify and remove the counter all together. It should be a simple update.
blocking-b2g: --- → 2.1?
Comment 3•10 years ago
|
||
If its a big UX improvement and simplify things we probably will uplift but we can't hold the release for this.
blocking-b2g: 2.1? → backlog
Updated•10 years ago
|
Priority: -- → P1
Assignee | ||
Comment 4•10 years ago
|
||
Attachment #8500344 -
Flags: ui-review?(epang)
Reporter | ||
Comment 5•10 years ago
|
||
Comment on attachment 8500344 [details]
Link to Pull Request: https://github.com/mozilla-b2g/gaia/pull/24818
Looks good, thanks Alberto!
Reporter | ||
Comment 6•10 years ago
|
||
Comment on attachment 8500344 [details]
Link to Pull Request: https://github.com/mozilla-b2g/gaia/pull/24818
would help if I R+ :)
Attachment #8500344 -
Flags: ui-review?(epang) → ui-review+
Reporter | ||
Comment 7•10 years ago
|
||
Hey Alberto, does this need to be reviewed? Hope this can happen soon so we can flag for uplift.
Flags: needinfo?(apastor)
Assignee | ||
Updated•10 years ago
|
Attachment #8500344 -
Flags: review?(etienne)
Flags: needinfo?(apastor)
Updated•10 years ago
|
Attachment #8500344 -
Flags: review?(etienne) → review+
Assignee | ||
Comment 8•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•10 years ago
|
status-b2g-v2.1:
--- → affected
status-b2g-v2.2:
--- → fixed
Assignee | ||
Comment 9•10 years ago
|
||
Comment on attachment 8500344 [details]
Link to Pull Request: https://github.com/mozilla-b2g/gaia/pull/24818
[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): Ambient indicator feature
[User impact] if declined: The user will still see the notifications counter on the ambient indicator. See https://bugzilla.mozilla.org/show_bug.cgi?id=1073480#c2 for more details about why for this release.
[Testing completed]: Removing a feature, so just acommodating the unit tests. Deep UX review.
[Risk to taking this patch] (and alternatives if risky): Low risk, just removing a class (and the javascript that decides it).
[String changes made]: -
Attachment #8500344 -
Flags: approval-gaia-v2.1?(fabrice)
Comment 10•10 years ago
|
||
(In reply to Gregor Wagner [:gwagner] from comment #3)
> If its a big UX improvement and simplify things we probably will uplift but
> we can't hold the release for this.
Hi Gregor...
We had feedback that having the dual band of color in the indicator was confusing people as they thought the animation was "stuck" and the phone wasn't working. That's why ux is pushing to get this one through. Can we still request uplift?
Rob
Flags: needinfo?(anygregor)
Comment 11•10 years ago
|
||
(In reply to Rob MacDonald [:robmac] from comment #10)
> (In reply to Gregor Wagner [:gwagner] from comment #3)
> > If its a big UX improvement and simplify things we probably will uplift but
> > we can't hold the release for this.
>
> Hi Gregor...
>
> We had feedback that having the dual band of color in the indicator was
> confusing people as they thought the animation was "stuck" and the phone
> wasn't working. That's why ux is pushing to get this one through. Can we
> still request uplift?
>
> Rob
Alberto asked for approval in comment 9.
Flags: needinfo?(anygregor)
Comment 12•10 years ago
|
||
This issue is verified on Flame 2.2:
Flame 2.2 Master KK (319mb) (Full Flash)
Device: Flame 2.2 Master KK (319mb) (Full Flash)
BuildID: 20141012040203
Gaia: 717ad4e8b7fc10ab8248500d00ba5ba0977fa8ab
Gecko: 44168a7af20d
Gonk: 52c909e821d107d414f851e267dedcd7aae2cebf
Version: 35.0a1 (2.2 Master)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:35.0) Gecko/35.0 Firefox/35.0
The count indicator is removed.
--------------------------------------------
Leaving verifyme keyword for 2.1.
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Keywords: verifyme
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Updated•10 years ago
|
Attachment #8500344 -
Flags: approval-gaia-v2.1?(fabrice) → approval-gaia-v2.1+
Comment 13•10 years ago
|
||
Target Milestone: --- → 2.1 S6 (10oct)
Comment 14•10 years ago
|
||
This issue is verified on Flame 2.1:
Device: Flame 2.1 KK (319mb) (Full Flash)
BuildID: 20141017001201
Gaia: 1ea74943cfe525c76a074ca1d7de8e51a70f6b98
Gecko: 2befa902ff5c
Gonk: 05aa7b98d3f891b334031dc710d48d0d6b82ec1d
Version: 34.0 (2.1)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
The count indicator is removed.
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Updated•10 years ago
|
blocking-b2g: backlog → ---
tracking-b2g:
--- → backlog
You need to log in
before you can comment on or make changes to this bug.
Description
•