828552 - [SMS] Fix transitions without flickering

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::SMS, defect, P3)

Description

[Borja Salguero [:borjasalguero]]

Currently the transition it's horrible in SMS App. We are going to restore the previous handling of transitions that it's smooth and reliable until Platform will be ready.

Comment 1

[Borja Salguero [:borjasalguero]]

Attachment: PR

Comment 2

[Borja Salguero [:borjasalguero]]

For me it should be considered bb+ due to the user experience it's not the best with a 'flickering' between transitions. This code could be reverted once the platform will be working with this type of transitions without flickering.

Comment 3

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

-moz-element has some very bad implications for the keyboard. I'm strongly against this hack.

Comment 4

[Borja Salguero [:borjasalguero]]

In this case I've tested the Keyboard and I've not realized about any big impact (I know that in other apps the impact of moz-element & Keyboard it's so bad, but this seems not be the case). However, as it's said in the bug, it's about restoring a workaround until the platform will be working in a right way with this type of animation, so we could remove it once there will be no 'flickering'. 
In this case we should analyze as a trade-off for V1, due to the user experience with this patch applied it's better than without it.

Comment 5

[Steve Chung [:steveck]]

Hi Vivien, Is there any serious drawback for applying -moz-element under the keyboard(rendering broken or performance decreasing)? BTW, if we can apply snapshot only for animation and hide it when transition end, could it be safer to use -moz-element in the app?

Comment 6

[David Scravaglieri [:scravag]]

We will wait for platform.

Comment 7

[Borja Salguero [:borjasalguero]]

Updating the name of this bug. It's related to 827382, so this one should be bb+ as well for being consistent.

Comment 8

[Borja Salguero [:borjasalguero]]

Attachment: PR

Comment 9

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

Comment on attachment 700415 [details]
PR

https://github.com/mozilla-b2g/gaia/commit/86549ba5af5de93302648a325ee75f1543459bac