912028 - [Keyboard] [New IME] Background is visible when keyboard pop in animation plays

Status: RESOLVED FIXED

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

Description

[Jan Jongboom [:janjongboom] (inactive)]

The client app gets new height immediately instead of when the keyboard pop in animation is ended causing the background image to be visible.

Easy to be seen in FF desktop but also reproducible on device.

Newly introduced with the new IME changes.

Comment 1

[Ivan Tsay (:ITsay)]

Need to be fixed.

Comment 2

[Jan Jongboom [:janjongboom] (inactive)]

Attachment: Patch

The current code tried to address this issue already but made two wrong assumptions:

If hide was not in classList of keyboard the height would be updated directly. It didn't take into account that the open transition might be happening at that time and the height would be incorrect.
Listens to first transform event that comes in, and that is the visibility transition; not the transform (which we should be waiting for).
(CSS) We ease on visibility but that doesn't have an ease. Should do it on opacity.
So those are the issues addressed. I've written tests around it because it felt quite fragile when touching it; and it helps with quickly testing my assumptions as well :-)

Comment 3

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

Comment on attachment 811057 [details] [review]
Patch

[Steal the review]

Looks good to me, except some minor comments on the test cases.

Jan,

Thanks for the good work and even adding the test cases.

Comment 4

[Jan Jongboom [:janjongboom] (inactive)]

Landed in https://github.com/mozilla-b2g/gaia/commit/ea8548e802138bb1830f6c8991d4beb461b6207c

Do we need to do uplift ourselves?

Comment 5

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

(In reply to Jan Jongboom [:janjongboom] from comment #4)
> Landed in
> https://github.com/mozilla-b2g/gaia/commit/
> ea8548e802138bb1830f6c8991d4beb461b6207c
> 
> Do we need to do uplift ourselves?

I guess not.
koi+ and setting status-b2g-v1.2 to "affected" would make this uplifted auto-magically. :)

Comment 6

[John Ford [:jhford] CET/CEST Berlin Time]

Uplifted ea8548e802138bb1830f6c8991d4beb461b6207c to:
v1.2: 8baf96d506e84af2e3ab0832062a7c81ba36df37