866643 - Homescreen is zoomed if an application close unexpectingly

Status: RESOLVED WORKSFORME

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

Description

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

I don't have a good steps to reproduce but I have seen some cases where the homescreen is zoomed in when it is the focused app. It seems to reproduce when an application is closed too quickly for the animation to finish.

Comment 1

[Cristian Rodriguez (:crdlc)]

Vivien, are you working on it? If you want I could take a look as well for helping you, thanks

Comment 2

[Cristian Rodriguez (:crdlc)]

Attachment: Patch v1

Comment 3

[Maria Angeles Oteo (:oteo)]

In which branch have you seen this bug?

Comment 4

[Cristian Rodriguez (:crdlc)]

Regression from bug 804270 that was only landed on master until right now

Comment 5

[Cristian Rodriguez (:crdlc)]

I wanted to fix it because I guess that open and close animations will be available on v1-train in the future

Comment 6

[Cristian Rodriguez (:crdlc)]

Hi Vivien, any update here? Thanks

Comment 7

[Cristian Rodriguez (:crdlc)]

Please Alive, could you take a look to this one? The review was requested in 2013-04-30 and I want to land this one in order to fix the problem. Thanks a lot my friend!!

Comment 8

[(inactive after 6/18) Alive Kuo [:alive]]

Honestly, I have to say I disagree to endlessly pollute Window Manager for a master-only feature. But I also have to apologize for the slow reaction on WM testable plan because I have lots of blocker bugs right now...

Transfer the review back to Vivien again.

Comment 9

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 743536 [details]
Patch v1

See my last comment.

Comment 10

[Cristian Rodriguez (:crdlc)]

Is Vivien on holidays?

Comment 11

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to Cristian Rodriguez de la Cruz (:crdlc) from comment #10)
> Is Vivien on holidays?

Lots of holidays in France during May. Also I feel like I fixed it in bug 866174. Instead of spreading the calls to add/remove the classes I have tried to move them all together.

Comment 12

[Cristian Rodriguez (:crdlc)]

ok, thanks, I am going to try to reproduce with the current master, if it works fine, I will close this bug. Thanks

Comment 13

[Cristian Rodriguez (:crdlc)]

After landing this bug 866174 I cannot reproduce the problem, thanks Vivien