542623 - don't dismiss menubar and sip button while focused on a text box

Status: RESOLVED FIXED

(Core :: Widget: Win32, defect)

Description

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: patch

with bug 532929 we show the sip button when the keyboard is displayed, but if you click the sip button to dismiss the keyboard the sip button also hides. This means there is no way to bring back the keyboard.

Also, on start up, we are currently dismissing the keyboard without taking the focus from the url bar. As a result, the only way to enter a url is to dismiss the awesome panel and reopen it by clicking on the url bar, which is less than obvious.

Comment 1

[Alex Pakhotin (:alexp)]

Comment on attachment 423852 [details] [diff] [review]
patch

>@@ -89,33 +90,30 @@ void nsWindowCE::OnSoftKbSettingsChange(
...
>     if (winRect.bottom != visRect->bottom) {
>-      if (winRect.bottom < visRect->bottom) {
>-        // Soft keyboard has been hidden, have to hide the SIP button as well
>-        if (sSoftKeyMenuBarHandle)
>-          ShowWindow(sSoftKeyMenuBarHandle, SW_HIDE);
>-
>-        SHFullScreen(wndMain, SHFS_HIDESIPBUTTON);
>-      }

I don't think deleting this code from here is correct.
Now with this change the SIP button frequently stays visible when it's supposed to hide - for example when an edit box loses its focus.

I tried different combinations and found that having this stuff in OnSoftKbSettingsChange rather than in ToggleSoftKB worked better for me. I cannot say for sure if it's the best solution though.

All this SIP stuff is a bit shaky - it works most of the time, but sometimes the same code in the same conditions behaves completely differently. It seems to depend on the timings of IM/SIP and other events received by the window.

Comment 2

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: patch v.2

you're right, since we're using the async shell function to raise and lower the softkb now, the keyboard hasn't made its move by the time we leave ToggleSoftKB() and when it does move it brings the SIP button up with it.

ToggleSoftKB() authoritatively knows when an editable element has focus and when it looses it (if that's wrong, our IME code is busted).  Therefore, that is where were should be determining where to show and hide the SIP button and menu bar.  This patch adds code to hide the SIP button back in to the OnSoftKbSettingsChange() if the static set by ToggleSoftKB() is true.  It also calls OnSoftKbSettingsChange() directly when hiding the keyboard since its possible that the keyboard is already hidden and we won't get an event through the message loop.

We don't have to worry about the menu bar in OnSoftKbSettingsChange() since the keyboard coming and going doesn't effect that.

Comment 3

[Alex Pakhotin (:alexp)]

Now the SIP button seems to be hiding properly.

But the autocomplete popup is resized incorrectly sometimes. See a screenshot here: http://people.mozilla.org/~alexp/542623-sip-menubar/resized-window.png
It happens for example when after dismissal of that popup opened on the start up,
you tap in the awesome bar the first time, and the keyboard appears.

Comment 4

[Alex Pakhotin (:alexp)]

Comment on attachment 423960 [details] [diff] [review]
patch v.2

OK, it works with SoftKeyboardObserver disabled in the browser.js.
We just need that patch in addition to this.

Comment 5

[Doug Turner (:dougt)]

Comment on attachment 423960 [details] [diff] [review]
patch v.2

okay; land together.

it would be helpful to have comments as to what you are doing.

Comment 6

[Brad Lassey [:blassey] (use needinfo?)]

pushed http://hg.mozilla.org/mozilla-central/rev/18672730444e

leaving bug open for the mobile-browser patch

Comment 7

[Brad Lassey [:blassey] (use needinfo?)]

sorry for the bug spam, the mobile-browser patch is actually on bug 542884, closing this one

Comment 8

[Brad Lassey [:blassey] (use needinfo?)]

pushed http://hg.mozilla.org/releases/mozilla-1.9.2/rev/aa7c5d7fd611