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)

x86
Gonk (Firefox OS)
defect

Tracking

(tracking-b2g:backlog, b2g-v2.1 verified, b2g-v2.2 verified)

VERIFIED FIXED
2.1 S6 (10oct)
tracking-b2g backlog
Tracking Status
b2g-v2.1 --- verified
b2g-v2.2 --- verified

People

(Reporter: epang, Assigned: apastor)

Details

(Keywords: polish, Whiteboard: [systemsfe])

Attachments

(3 files)

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!
[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?
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
Priority: -- → P1
Comment on attachment 8500344 [details] Link to Pull Request: https://github.com/mozilla-b2g/gaia/pull/24818 Looks good, thanks Alberto!
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+
Hey Alberto, does this need to be reviewed? Hope this can happen soon so we can flag for uplift.
Flags: needinfo?(apastor)
Attachment #8500344 - Flags: review?(etienne)
Flags: needinfo?(apastor)
Attachment #8500344 - Flags: review?(etienne) → review+
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
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)
(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)
(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)
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
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Attachment #8500344 - Flags: approval-gaia-v2.1?(fabrice) → approval-gaia-v2.1+
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.
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Keywords: verifyme
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
blocking-b2g: backlog → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: