1018590 - While keyboard is present, swipe the edge to swipe app without actual switching between apps results hidden keyboard but focus not removed

Status: RESOLVED FIXED

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

Description

[Tim Guan-tin Chien [:timdream] (please needinfo)]

+++ This bug was initially created as a clone of Bug #1011087 +++

STR:

0. Enable sheet swipe
1. Open one app
2. Open another, find a input and focus it. Wait for keyboard
3. Start swiping from the edge without lift the finger
4. Move your finger back to the edge and lift.

Expected:

1. Keyboard being hidden on step 3, and focus removed

Actual:

1. Keyboard being hidden, but focus not removed

Note:

I think this is a incomplete fix of bug 1011087. We should remove the focus there. We don't run into this for a complete swipe because the focus will be removed by setVisible(false) in app window manager.

I recommend we try mozInputMethod.mgmt.removeFocus() if that works.

Triage: should block because incomplete work in bug 1011087.

Comment 1

[Etienne Segonzac (:etienne)]

Attachment: Gaia PR

Thanks Tim! Your suggestion worked like a charm.
Mind reviewing it? :)

Comment 2

[Tim Guan-tin Chien [:timdream] (please needinfo)]

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

I would have to rebase bug 1017327 but I think this 2.0+ should land first anyway.

Comment 3

[Etienne Segonzac (:etienne)]

https://github.com/mozilla-b2g/gaia/commit/ec9d4679cc55306b2efa9486c32f4f7c2adee35b