Shell needs to unfocus app when greeter is shown

[Bill Filler]

When the greeter is shown, either from screen timeout trigger or user pressing the power button, the shell needs to unfocus the currently focused app. Otherwise, if the app has focus and the keyboard is displayed it will stay displayed when the greeter is shown on power button event.

According to ricmm, the following needs to be done:
Shell needs to call unfocusCurrentApplication() when greeter.show(), but also refocus the index of the unfocused application when greeter.hide(), because otherwise the application that comes to view will be suspended

ricmm:

This comes in addition to a QtUbuntu branch that will cause a dismissal of the QInputPanel in the delegate for transition to the SUSPENDED state, effectively dismissing the keyboard and removing input focus from any active text field.

[Michał Sawicz (saviq)]

@ricmm I had a quick look and calling unfocusCurrentApplication() nor unfocusCurrentApplication(ApplicationInfo::MainStage) actually did unfocus the app (it was still there in ApplicationManager.mainStageFocusedApplication).

Any ideas?

[Ricardo Mendoza (ricmm)]

@saviq The reason for that is that we indeed do *not* support an unfocusCurrentApplication() method on the platform API. That naming decision comes from QtUbuntu that wraps it around unfocus_running_sessions() which just brings up focus (in the sense of layer visibility and input windows) to system-sessions only.

As I said in my comment in the description, the naming of this bug might be misleading. The right way is to dismiss input contexts when an application is about to be unfocused (either by means of returning to shell, or focusing another application). This is done via a delegate call, now that the new API code has landed I will propose a branch for this today.

[Michał Sawicz (saviq)]

(I'm off until Monday, btw...)

Hey ricmm, so what is it that we need to do in the shell still? I have a preliminary diff that unfocuses (but not refocuses yet) the app on greeter shown:

=== modified file 'Shell.qml'
--- Shell.qml 2013-05-28 08:38:45 +0000
+++ Shell.qml 2013-05-31 08:30:45 +0000
@@ -353,7 +353,12 @@
         width: parent.width
         height: parent.height - panel.panelHeight

- onShownChanged: if (shown) greeter.forceActiveFocus()
+ onShownChanged: {
+ if (shown) {
+ greeter.forceActiveFocus()
+ applicationManager.unfocusCurrentApplication();
+ }
+ }

         onUnlocked: greeter.hide()
         onSelected: {

[PS Jenkins bot (ps-jenkins)]

Fix committed into lp:unity/phablet at revision 721, scheduled for release in unity, milestone 13.05