Closed Bug 1069997 Opened 6 years ago Closed 6 years ago
The system app is constantly restyling when an update is available
The root cause is bug 962594, but the animation in question is actually *never* displayed since the utility tray was visually refreshed. Patch incoming.
This patch is implementing the progress to spec and by the same occasion fixing the constant restyle.
Assignee: nobody → etienne
Attachment #8492235 - Flags: review?(21)
currently on master 2.1 while the update is downloading the notification doesn't show anything. This patch is adding it according to https://mozilla.app.box.com/s/yek8hkezu40yjwix4dca, we don't have a progress value so we're displaying the "looping" indicator.
Attachment #8492238 - Flags: ui-review?(epang)
[Blocking Requested - why for this release]: Details can be found on bug 962594, but to sum up: when an update (system update or app update) is availablem the system app will never be idle (constant restyle) and will drain the battery. Worse, the element causing this is never displayed on screen.. And we have no activity indicator while downloading an update so we're not matching the spec. Thankfully, a small CSS patch fixes all of this, I'd ask for approval but I think we should block here because of the power consumption issue.
blocking-b2g: --- → 2.1?
Attachment #8492235 - Flags: review?(21) → review+
Major power issue.
blocking-b2g: 2.1? → 2.1+
(In reply to Etienne Segonzac (:etienne) from comment #2) > Created attachment 8492238 [details] > 2014-09-19-18-18-42.png > > currently on master 2.1 while the update is downloading the notification > doesn't show anything. This patch is adding it according to > https://mozilla.app.box.com/s/yek8hkezu40yjwix4dca, we don't have a progress > value so we're displaying the "looping" indicator. Hey Etienne, this looks good, but while updating the progress indicator for browser we found that an animated GIF worked better. Here's a link to the gifs: https://mozilla.box.com/s/o9y1vczu97ehfwtpgnhz Can we update so they match? Thanks!
I'm pretty sure we're already using this gif :)
(In reply to Etienne Segonzac (:etienne) from comment #6) > I'm pretty sure we're already using this gif :) I meant we're using the only one present in the system app.
Comment on attachment 8492238 [details] 2014-09-19-18-18-42.png great, in that case this is good :). Thanks Etienne!
Attachment #8492238 - Flags: ui-review?(epang) → ui-review+
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Comment on attachment 8492235 [details] [review] Gaia PR [Approval Request Comment] [Bug caused by] (feature/regressing bug #): the update manager, here since 1.0 [User impact] if declined: poorer battery life [Testing completed]: checked that the constant restyle is fixed, checked that the system update notification is still working properly, checked that the new progress indication is displaying properly [Risk to taking this patch] (and alternatives if risky): low [String changes made]: zilch
Attachment #8492235 - Flags: approval-gaia-v2.1?
Attachment #8492235 - Flags: approval-gaia-v2.1? → approval-gaia-v2.1+
verifyme on 2.1 branch
Verified fixed on Flame 2.1 and Flame 2.2. Result: Download progress indicator is displayed while downloading an update. Flame 2.1 Device: Flame 2.1 (319mb)(Kitkat Base)(Full Flash) BuildID: 20141024001204 Gaia: 0f76e0baac733cca56d0140e954c5f446ebc061f Gecko: 7d78ff7d25b6 Gonk: 05aa7b98d3f891b334031dc710d48d0d6b82ec1d Version: 34.0 (2.1) Firmware: V188 User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0 Flame 2.2 Device: Flame Master (319mb)(KitKat Base)(FullFlash) BuildID: 20141023040204 Gaia: 27a1d1baaa8e375b70e043efee67d5f2206c330b Gecko: 88adcf8fef83 Gonk: 05aa7b98d3f891b334031dc710d48d0d6b82ec1d Version: 36.0a1 (2.2 Master) Firmware: V188 User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
You need to log in before you can comment on or make changes to this bug.