Merge lp:~renatofilho/address-book-app/input-hints2 into lp:address-book-app

Proposed by Renato Araujo Oliveira Filho
Status: Merged
Approved by: Bill Filler
Approved revision: 55
Merged at revision: 64
Proposed branch: lp:~renatofilho/address-book-app/input-hints2
Merge into: lp:address-book-app
Diff against target: 132 lines (+31/-1)
6 files modified
src/imports/Common/ContactDetailGroupBase.qml (+24/-0)
src/imports/ContactEdit/ContactDetailEmailsEditor.qml (+1/-0)
src/imports/ContactEdit/ContactDetailGroupWithTypeEditor.qml (+2/-0)
src/imports/ContactEdit/ContactDetailOnlineAccountsEditor.qml (+1/-0)
src/imports/ContactEdit/ContactDetailPhoneNumbersEditor.qml (+1/-0)
src/imports/ContactEdit/ContactDetailWithTypeEditor.qml (+2/-1)
To merge this branch: bzr merge lp:~renatofilho/address-book-app/input-hints2
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Bill Filler (community) Approve
Tiago Salem Herrmann Pending
Review via email: mp+185179@code.launchpad.net

This proposal supersedes a proposal from 2013-09-06.

Commit message

fix focus on creation

Description of the change

merge latest from trunk

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
Tiago Salem Herrmann (tiagosh) wrote : Posted in a previous version of this proposal

Using Qt.ImhDialableCharactersOnly prevents the user to add some common phone number characters, like parentheses, comma and hyphen. Also, the button '+*' in the osk does not work at all.

review: Needs Information
Revision history for this message
Renato Araujo Oliveira Filho (renatofilho) wrote : Posted in a previous version of this proposal

> Using Qt.ImhDialableCharactersOnly prevents the user to add some common phone
> number characters, like parentheses, comma and hyphen. Also, the button '+*'
> in the osk does not work at all.

I do not think the parentheses, comma and hyphen should be accepted. Since they are not DialableCharacters. As the enum says.

About the "+*" there is a related bug report on Qt: https://bugreports.qt-project.org/browse/QTBUG-13696

Revision history for this message
Tiago Salem Herrmann (tiagosh) wrote : Posted in a previous version of this proposal

In general we normalize the phone numbers before making calls. So the non diable characters wouldn't be a problem.
It is commom to use () for area codes. IMO we should allow users to enter those chars
Also, comma should be considered as diable, it means to stop a second before dialing the next numbers, used mostly to call extensions IIRC.

Revision history for this message
Bill Filler (bfiller) wrote : Posted in a previous version of this proposal

I think the MR is fine, but the OSK layout is totally broken for phone numbers, both in layout and certain characters not working. Lets hold off on landing this until the OSK is fixed.

review: Abstain
Revision history for this message
Bill Filler (bfiller) wrote : Posted in a previous version of this proposal

btw, the email layout seems ok but not sure it's worth reverting the phone number code or just wait till osk fixed

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:54
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~renatofilho/address-book-app/input-hints2/+merge/185179/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/address-book-app-ci/138/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-saucy-amd64-ci/138
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-saucy-armhf-ci/139
        deb: http://jenkins.qa.ubuntu.com/job/address-book-app-saucy-armhf-ci/139/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-saucy-i386-ci/138
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-saucy/3312
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-touch/851
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-i386/3320
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-i386/3320/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-saucy/2800
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/853
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/853/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/710
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/720

Click here to trigger a rebuild:
http://s-jenkins:8080/job/address-book-app-ci/138/rebuild

review: Needs Fixing (continuous-integration)
55. By Renato Araujo Oliveira Filho

Avoid fields to get focus during the creation.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:55
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~renatofilho/address-book-app/input-hints2/+merge/185179/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/address-book-app-ci/139/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-saucy-amd64-ci/139
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-saucy-armhf-ci/140
        deb: http://jenkins.qa.ubuntu.com/job/address-book-app-saucy-armhf-ci/140/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-saucy-i386-ci/139
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-saucy/3315
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-touch/853
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-i386/3323
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-i386/3323/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-saucy/2803
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/855
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/855/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/712
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/722

Click here to trigger a rebuild:
http://s-jenkins:8080/job/address-book-app-ci/139/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Bill Filler (bfiller) wrote :

works well, landing

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/imports/Common/ContactDetailGroupBase.qml'
--- src/imports/Common/ContactDetailGroupBase.qml 2013-07-26 00:19:09 +0000
+++ src/imports/Common/ContactDetailGroupBase.qml 2013-09-11 23:55:19 +0000
@@ -31,6 +31,7 @@
31 property alias headerDelegate: headerItem.sourceComponent31 property alias headerDelegate: headerItem.sourceComponent
32 property Component detailDelegate32 property Component detailDelegate
33 property int minimumHeight: 033 property int minimumHeight: 0
34 property bool loaded: false
3435
35 implicitHeight: root.details.length > 0 ? contents.height : minimumHeight36 implicitHeight: root.details.length > 0 ? contents.height : minimumHeight
36 visible: implicitHeight > 037 visible: implicitHeight > 0
@@ -45,6 +46,7 @@
45 onValuesChanged: {46 onValuesChanged: {
46 if (!values) {47 if (!values) {
47 clear()48 clear()
49 root.loaded = false
48 return50 return
49 }51 }
5052
@@ -61,6 +63,10 @@
61 set(i, {"detail": values[i]})63 set(i, {"detail": values[i]})
62 }64 }
63 }65 }
66
67 if (!root.loaded) {
68 loadTimer.restart()
69 }
64 }70 }
65 }71 }
6672
@@ -92,6 +98,24 @@
92 value: root.details[index]98 value: root.details[index]
93 }99 }
94 }100 }
101
102 onItemAdded: {
103 if (root.loaded) {
104 item.forceActiveFocus()
105 }
106 }
107
95 }108 }
96 }109 }
110
111 // This timer will help to avoid fields get focus during the page creation
112 // At the first time that the page is loaded the timer will start and all
113 // subsequent details added before the timeout would not receive focus,
114 // after the timeout the new fields will receive focus as default
115 Timer {
116 id: loadTimer
117
118 interval: 500
119 onTriggered: root.loaded = true
120 }
97}121}
98122
=== modified file 'src/imports/ContactEdit/ContactDetailEmailsEditor.qml'
--- src/imports/ContactEdit/ContactDetailEmailsEditor.qml 2013-07-25 21:16:28 +0000
+++ src/imports/ContactEdit/ContactDetailEmailsEditor.qml 2013-09-11 23:55:19 +0000
@@ -25,4 +25,5 @@
25 detailType: QtContacts.ContactDetail.Email25 detailType: QtContacts.ContactDetail.Email
26 fields: [ 0 ]26 fields: [ 0 ]
27 placeholderTexts: [ i18n.tr("Enter an email address") ]27 placeholderTexts: [ i18n.tr("Enter an email address") ]
28 inputMethodHints: Qt.ImhEmailCharactersOnly
28}29}
2930
=== modified file 'src/imports/ContactEdit/ContactDetailGroupWithTypeEditor.qml'
--- src/imports/ContactEdit/ContactDetailGroupWithTypeEditor.qml 2013-07-26 00:19:09 +0000
+++ src/imports/ContactEdit/ContactDetailGroupWithTypeEditor.qml 2013-09-11 23:55:19 +0000
@@ -28,6 +28,7 @@
28 property int currentItem: -128 property int currentItem: -1
29 property int fieldType: QtContacts.ContactDetail.FieldContext29 property int fieldType: QtContacts.ContactDetail.FieldContext
30 property variant placeholderTexts: []30 property variant placeholderTexts: []
31 property variant inputMethodHints
3132
32 function save() {33 function save() {
33 var changed = false34 var changed = false
@@ -137,6 +138,7 @@
137 height: implicitHeight138 height: implicitHeight
138 width: root.width139 width: root.width
139140
141 inputMethodHints: root.inputMethodHints
140 onDetailChanged: updateCombo(false)142 onDetailChanged: updateCombo(false)
141143
142 // this is necessary due the default property of ListItem.Empty144 // this is necessary due the default property of ListItem.Empty
143145
=== modified file 'src/imports/ContactEdit/ContactDetailOnlineAccountsEditor.qml'
--- src/imports/ContactEdit/ContactDetailOnlineAccountsEditor.qml 2013-07-31 19:36:55 +0000
+++ src/imports/ContactEdit/ContactDetailOnlineAccountsEditor.qml 2013-09-11 23:55:19 +0000
@@ -27,4 +27,5 @@
27 fields: [ QtContacts.OnlineAccount.AccountUri ]27 fields: [ QtContacts.OnlineAccount.AccountUri ]
28 placeholderTexts: [ i18n.tr("Enter a social alias") ]28 placeholderTexts: [ i18n.tr("Enter a social alias") ]
29 typeModel: ContactDetailOnlineAccountTypeModel { }29 typeModel: ContactDetailOnlineAccountTypeModel { }
30 inputMethodHints: Qt.ImhEmailCharactersOnly
30}31}
3132
=== modified file 'src/imports/ContactEdit/ContactDetailPhoneNumbersEditor.qml'
--- src/imports/ContactEdit/ContactDetailPhoneNumbersEditor.qml 2013-07-31 19:36:55 +0000
+++ src/imports/ContactEdit/ContactDetailPhoneNumbersEditor.qml 2013-09-11 23:55:19 +0000
@@ -27,4 +27,5 @@
27 fields: [ QtContacts.PhoneNumber.Number ]27 fields: [ QtContacts.PhoneNumber.Number ]
28 placeholderTexts: [ i18n.tr("Enter a number") ]28 placeholderTexts: [ i18n.tr("Enter a number") ]
29 typeModel: ContactDetailPhoneNumberTypeModel { }29 typeModel: ContactDetailPhoneNumberTypeModel { }
30 inputMethodHints: Qt.ImhDialableCharactersOnly
30}31}
3132
=== modified file 'src/imports/ContactEdit/ContactDetailWithTypeEditor.qml'
--- src/imports/ContactEdit/ContactDetailWithTypeEditor.qml 2013-08-28 17:06:38 +0000
+++ src/imports/ContactEdit/ContactDetailWithTypeEditor.qml 2013-09-11 23:55:19 +0000
@@ -29,6 +29,7 @@
29 property int fieldType: -129 property int fieldType: -1
30 property alias selectedTypeIndex: detailTypeSelector.currentIndex30 property alias selectedTypeIndex: detailTypeSelector.currentIndex
31 property variant placeholderTexts: []31 property variant placeholderTexts: []
32 property var inputMethodHints
3233
33 function selectType(type) {34 function selectType(type) {
34 detailTypeSelector.selectItem(type)35 detailTypeSelector.selectItem(type)
@@ -99,13 +100,13 @@
99 Component.onCompleted: {100 Component.onCompleted: {
100 if (index == 0) {101 if (index == 0) {
101 focus = true102 focus = true
102 detail.forceActiveFocus()
103 }103 }
104 }104 }
105 focus: false105 focus: false
106 detail: root.detail106 detail: root.detail
107 field: modelData107 field: modelData
108 placeholderText: root.placeholderTexts[index]108 placeholderText: root.placeholderTexts[index]
109 inputMethodHints: root.inputMethodHints
109110
110 anchors {111 anchors {
111 left: parent.left112 left: parent.left

Subscribers

People subscribed via source and target branches