Merge lp:~bzoltan/ubuntu-ui-toolkit/inputMainFocus into lp:ubuntu-ui-toolkit

[Zoltan Balogh]

Text input focus must always follow main component. New tests assert that the OSK is shown. Fixes LP: #1545802

Text input focus must always follow main component. New tests assert that the OSK is shown. Fixes LP: #1545802

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1315
http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-ci/2321/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-vivid-amd64-ci/144/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-vivid-armhf-ci/145/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-ui-toolkit-ci/2321/rebuild

review: Needs Fixing (continuous-integration)

Unmerged revisions

1315. By Zoltan Balogh on 2016-05-11

Text input focus must always follow main component. New tests assert that the OSK is shown. Fixes LP: #1545802

=== modified file 'src/Ubuntu/Components/1.3/InputHandler.qml'
--- src/Ubuntu/Components/1.3/InputHandler.qml	2016-04-25 10:45:41 +0000
+++ src/Ubuntu/Components/1.3/InputHandler.qml	2016-05-11 09:15:23 +0000
@@ -292,9 +292,7 @@
         target: main
         ignoreUnknownSignals: true
         onActiveFocusChanged: {
-            if (!main.activeFocus) {
-                input.focus = false;
-            }
+            input.focus = main.activeFocus;
         }
         onKeyNavigationFocusChanged: {
             if (main.keyNavigationFocus) {
 
=== modified file 'tests/unit_x11/tst_components/tst_textinput_common13.qml'
--- tests/unit_x11/tst_components/tst_textinput_common13.qml	2016-04-08 07:40:38 +0000
+++ tests/unit_x11/tst_components/tst_textinput_common13.qml	2016-05-11 09:15:23 +0000
@@ -23,7 +23,7 @@
 Item {
     id: testMain
     width: units.gu(40)
-    height: units.gu(50)
+    height: units.gu(70)
 
     Component {
         id: popoverComponent
@@ -97,6 +97,32 @@
             id: textField
         }
 
+        FocusScope {
+            id: scope
+            width: textFieldInMouseArea.implicitWidth
+            height: textFieldInMouseArea.implicitHeight
+            TextField {
+                anchors.fill: parent
+                id: textFieldInMouseArea
+                text: 'Lorem ipsum dolor sit amet'
+                color: UbuntuColors.blue
+            }
+            MouseArea {
+                anchors.fill: parent
+                enabled: !scope.activeFocus
+                onClicked: {
+                    textFieldInMouseArea.forceActiveFocus()
+                    textFieldInMouseArea.selectAll()
+                }
+                Rectangle {
+                    anchors.fill: parent
+                    color: UbuntuColors.blue
+                    opacity: 0.1
+                    visible: parent.enabled
+                }
+            }
+        }
+
         TextField {
             id: customTextField
             text: 'Lorem ipsum dolor sit amet'
@@ -198,6 +224,11 @@
             cursorRectSpy.clear();
             scrollerSpy.clear();
             escapePressedSpy.clear();
+            // Hide OSK if showing
+            UbuntuApplication.inputMethod.visible = false;
+            // Dismiss popover if any
+            mouseClick(testMain, 1, 1);
+            wait(500);
         }
 
         function test_context_menu_items_data() {
@@ -543,5 +574,22 @@
             keyClick(Qt.Key_Escape);
             compare(escapePressedSpy.count, 1);
         }
+
+        function test_a_text_field_evokes_osk_data() {
+            return [
+                { tag: 'textField', input: textField },
+                { tag: 'textField with icons', input: customTextField },
+                { tag: 'textArea', input: textArea },
+                { tag: 'focusScope', input: textFieldInMouseArea },
+            ];
+        }
+        function test_a_text_field_evokes_osk(data) {
+            waitForRendering(data.input);
+            compare(data.input.activeFocus, false, 'TextField is not yet focused');
+            mouseClick(data.input);
+            waitForRendering(data.input);
+            compare(data.input.activeFocus, true, 'TextField is focused');
+            compare(UbuntuApplication.inputMethod.visible, true, 'OSK is visible');
+        }
     }
 }