Merge lp:~mzanetti/reminders-app/fix-textfields into lp:reminders-app

[Michael Zanetti]

fix predictive text input

[Nekhelesh Ramananthan (nik90)]

LGTM! InputMethodComposing helps detect if there is partial text entered (due to predictive text enabled) and should work as expected.

review: Approve (code-review)

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

PASSED: Continuous integration, rev:476
http://91.189.93.70:8080/job/reminders-app-ci/782/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/reminders-app-vivid-amd64-ci/204

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/reminders-app-ci/782/rebuild

review: Approve (continuous-integration)

[Riccardo Padovani (rpadovani)]

lgtm, thanks

review: Approve

=== modified file 'src/app/qml/components/EditTagsDialog.qml'
--- src/app/qml/components/EditTagsDialog.qml	2015-09-10 22:50:16 +0000
+++ src/app/qml/components/EditTagsDialog.qml	2015-09-10 22:50:16 +0000
@@ -98,7 +98,7 @@
                 border.width: units.dp(1)
                 border.color: "black"
                 height: Math.min(5, tagsListView.count) * units.gu(4)
-                visible: textField.text.length > 0 && (textField.focus || tagsListView.focus)
+                visible: (textField.text.length > 0 || textField.inputMethodComposing) && (textField.focus || tagsListView.focus)
 
                 ListView {
                     id: tagsListView

=== modified file 'src/app/qml/ui/NotebooksPage.qml'
--- src/app/qml/ui/NotebooksPage.qml	2015-07-25 01:52:22 +0000
+++ src/app/qml/ui/NotebooksPage.qml	2015-09-10 22:50:16 +0000
@@ -207,7 +207,7 @@
 
             Button {
                 text: i18n.tr("OK")
-                enabled: nameTextField.text
+                enabled: nameTextField.text || nameTextField.inputMethodComposing
                 onClicked: {
                     renameNotebookDialog.accepted(nameTextField.text)
                     PopupUtils.close(renameNotebookDialog)