846525 - No animation when moving an application being downloaded

Status: RESOLVED DUPLICATE of bug 840160

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

Description

[Nicolas B. Pierron [:nbp]]

STR:
 - Go to the webpage of an application which install a lot & slowly.
 - Accept install and go to the home screen.
 - While the downloading icon is spinning, hold press on it.
 - then move it to another location.

Expected:
 - The dowloading icon should be scaled as other icon, and appear bigger under the finger.

Seen:
 - The icon only switch position when we reach the ending position instead of following the finger.

Comment 1

[Cristian Rodriguez (:crdlc)]

I have detected that the problem is more serious. I can reproduce that If I try to move the app that is downloading the homescreen die

Comment 2

[Cristian Rodriguez (:crdlc)]

Attachment: Patch v1

Comment 3

[Julien Wajsberg [:julienw]]

Ah, I was working last week on Bug 840160 which actually is the same bug.

Instead of adding a new state, and a new if block, I rather wanted to make that the downloading icon is displayed the same way as other icons, and so it would have a renderedIcon like the other. 

The CSS change is fine though !

Is it ok for you if we duplicate this bug to Bug 840160 ? I added some unit tests there; and I also added a "protection" which registers the "touchstart" event when we exit edit mode. I plan to change this bit to register the "touchstart" event everytime the user use the home button. Also, I don't completely get why we remove the "touchstart" event when we begin to drag so I'd like to investigate this too.

Comment 4

[Michael Vines [:m1] [:evilmachines]]

(In reply to crdlc from comment #1)
> I have detected that the problem is more serious. I can reproduce that If I
> try to move the app that is downloading the homescreen die

Hey crdlc, what does "homescreen die" mean?

Comment 5

[Julien Wajsberg [:julienw]]

It means "you can't do anything with the homescreen: pan, launch apps, etc, until you reboot".

Comment 6

[Julien Wajsberg [:julienw]]

If this is made tef+ we could try to land with Cristian's simpler and thus safer patch here, and go on with a more robust patch in Bug 840160 for master/v1.1.

Comment 7

[Cristian Rodriguez (:crdlc)]

OK Julien, you can duplicate this bug but remember to add the CSS change as well. Thanks

Comment 8

[Julien Wajsberg [:julienw]]

will have a patch ready today for bug 840160, I'll ask you the review !