1011087 - [B2G][SMS]Using edge gestures to change apps while keyboard is present, message bar is partially offscreen after returning to SMS app

Status: RESOLVED FIXED

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

Description

[Ross Kuhlman ([e-mail])]

Attachment: MessageBarLog.txt

Edge gestures can be used to cycle between open apps. If the messaging app is currently open and the keyboard is displayed, the messaging app will not display appropriately when the user swipes to a different app and back to messaging. The text bar at the bottom of the screen will be partially offscreen.

Repro Steps:
1) Update Open_C to Build ID: 20140515040207
2) Launch an app that does not support keyboard (Camera, Gallery, Music, etc)
3) Launch messaging app
4) Tap on a text field to summon keyboard
5) Use edge gesture to switch to app launched in step 2
6) Use edge gesture to switch back to messaging

Actual:
The 'Message' text box at the bottom of the screen is partially offscreen

Expected:
The 'Message' text box is entirely onscreen

Environmental Variables
Device: Open_C Master M-C
Build ID: 20140515040207
Gecko: e4843f4f08a7
Gaia: 3a1d67246a79e3632c3b3f2460a25291e7e2714c
Platform Version: 32.0a1
Firmware Version: P821A10 v1.0.0B06_LOG_DL

Notes:
Repro frequency: 100%
See attached: screenshot, logcat

Comment 1

[Ross Kuhlman ([e-mail])]

Attachment: MessageBarOffscreen

Comment 2

[Jason Smith [:jsmith]]

Can we get a video?

Comment 3

[Etienne Segonzac (:etienne)]

Attachment: Gaia PR

If we end up without a SheetTransition this will be moved pretty easily.
Until then I think this is the best place to put it.

Comment 4

[Marcia Knous [:marcia]]

Attachment: Screenshot of calendar app after using edge gestures

Etienne: I was about to file a bug, but is the attached screenshot part of the same issue and will your patch address it? Otherwise I can file a new bug

Comment 5

[Jason Smith [:jsmith]]

needinfo for comment 4

Comment 6

[Etienne Segonzac (:etienne)]

(In reply to Marcia Knous [:marcia - use needinfo] from comment #4)
> Created attachment 8423389 [details]
> Screenshot of calendar app after using edge gestures
> 
> Etienne: I was about to file a bug, but is the attached screenshot part of
> the same issue and will your patch address it? Otherwise I can file a new bug

If the keyboard was not involve in reproducing this screenshot we probably need a new bug :/

Comment 7

[Jason Smith [:jsmith]]

Removing regression keyword since this only happens if edge gestures is actually used.

Comment 8

[Natalya Kot [:nkot]]

Video of the issue - https://www.youtube.com/watch?v=Qd5XwId9zd0

Comment 9

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

Comment on attachment 8423368 [details] [review]
Gaia PR

r+ though I prefer an event for keyboard to close it but anyway...

Comment 10

[Etienne Segonzac (:etienne)]

Comment on attachment 8423368 [details] [review]
Gaia PR

Yeah.. it was already kinda ugly, switched to an event, can you take another quick look?

Comment 11

[Etienne Segonzac (:etienne)]

https://github.com/mozilla-b2g/gaia/commit/31ca6daae442ceacf942c74e747618deaff9c648

Comment 12

[Jason Smith [:jsmith]]

This is a bug in a preffed on feature, which should be using the blocking-b2g flag, not the feature-b2g flag.