Closed Bug 815206 Opened 12 years ago Closed 12 years ago

[homescreen] displays a confirmation and restarts a failed download

Categories

(Firefox OS Graveyard :: Gaia::Homescreen, defect, P1)

Other
Gonk (Firefox OS)
defect

Tracking

(blocking-basecamp:+)

RESOLVED FIXED
B2G C2 (20nov-10dec)
blocking-basecamp +

People

(Reporter: julienw, Assigned: arcturus)

References

Details

(Keywords: feature, late-l10n)

Attachments

(1 file)

In Bug 802596, we want to restart a failed Webapp installation.

- homescreen displays the confirmation screen to the user
- homescreen calls |download| on the app object
Nominating BB+ because depending Bug 802596 is BB+
Assignee: nobody → francisco.jordano
Blocks: 802596
blocking-basecamp: --- → ?
blocking-basecamp: ? → +
Priority: -- → P1
Target Milestone: --- → B2G C2 (20nov-10dec)
Francisco, how long do you estimate it will take for your fix for this?
Keywords: feature
Dietrich, hopefully the end of this week.
Josh, we need your wisdom here.

When the user accepts the restart confirmation, nothing changes on the homescreen until the first progress event is sent (ie until the first data is received).

What it means is that on a shitty network (like right now) the user would not have any notification that something happens. Moreover as nothing changed, he could still tap on the icon and would have the same restart confirmation window.

My view is that the icon should change as soon as the user accepts the restart and not react to taps anymore.
Flags: needinfo?(jcarpenter)
Please note that we are still pending Bug 815206 from platform to land.
Attachment #686706 - Flags: review?(felash)
Attachment #686706 - Flags: review?(crdlc)
(In reply to Francisco Jordano [:arcturus] from comment #5)
> Please note that we are still pending Bug 815206 from platform to land.

Nice try. I only clicked through that cycle 32 times before catching on.
(In reply to Dietrich Ayala (:dietrich) from comment #6)
> (In reply to Francisco Jordano [:arcturus] from comment #5)
> > Please note that we are still pending Bug 815206 from platform to land.
> 
> Nice try. I only clicked through that cycle 32 times before catching on.

LoL:

I mean, it's pending
Depends on: 815547
(In reply to Francisco Jordano [:arcturus] from comment #7)
> (In reply to Dietrich Ayala (:dietrich) from comment #6)
> > (In reply to Francisco Jordano [:arcturus] from comment #5)
> > > Please note that we are still pending Bug 815206 from platform to land.
> > 
> > Nice try. I only clicked through that cycle 32 times before catching on.
> 
> LoL:
> 
I mean, it's pending 815547, it's a nice trick, will use it often.

Tried the platform patch locally and is looking good :) with this + the changes that Julien did to the system app
Comment on attachment 686706 [details]
Pointer to GitHub PR 6722

reviewed on github, didn't try the patch yet

I don't know how the homescreen works so take my comments as suggestions and not as hard problems.
Comment on attachment 686706 [details]
Pointer to GitHub PR 6722

found a blocker: the icon is not updated when we receive a downloaderror or downloadsuccess after a restart.
Attachment #686706 - Flags: review?(felash) → review-
and we still need Josh's answer...
Depends on: 816535
(In reply to Julien Wajsberg [:julienw] from comment #4)
> My view is that the icon should change as soon as the user accepts the
> restart and not react to taps anymore.

Agreed. Correct me if I'm wrong, but we're talking about a programming change that is required in order to ensure faithfulness to the spec?
Flags: needinfo?(jcarpenter)
(In reply to Josh Carpenter [:jcarpenter] from comment #12)
> 
> Agreed. Correct me if I'm wrong, but we're talking about a programming
> change that is required in order to ensure faithfulness to the spec?

Yep (but the spec was not so clear about this).

Thanks !
Keywords: late-l10n
Francisco> what do you think about landing this bug without Josh's changes, and opening a new bug for tracking this ?

Or do you want to do it all at once ?
:julienw i'm adding the changes, when the user clicks we will show the spinner.

Right now just testing the edge cases, will ping you for a second review.

Thanks a lot!
Attachment #686706 - Flags: review?(crdlc)
Comment on attachment 686706 [details]
Pointer to GitHub PR 6722

let's do a follow up for apps with more than one entry point because I think we'll have some visual problems (although the feature should still work).
Attachment #686706 - Flags: review- → review+
Thanks, definitely we need a follow up for all the weird cases that we could find.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: