Merge lp:~renatofilho/address-book-app/fix-1233785 into lp:address-book-app

Proposed by Renato Araujo Oliveira Filho
Status: Merged
Merged at revision: 140
Proposed branch: lp:~renatofilho/address-book-app/fix-1233785
Merge into: lp:address-book-app
Diff against target: 53 lines (+6/-3)
3 files modified
src/imports/ContactEdit/ContactDetailWithTypeEditor.qml (+3/-2)
src/imports/ContactEdit/TextInputDetail.qml (+1/-0)
src/imports/ContactEdit/ValueSelector.qml (+2/-1)
To merge this branch: bzr merge lp:~renatofilho/address-book-app/fix-1233785
Reviewer Review Type Date Requested Status
Bill Filler (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+205270@code.launchpad.net

Commit message

Handle read-only fields.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Bill Filler (bfiller) wrote :

approved

review: Approve
Revision history for this message
Renato Araujo Oliveira Filho (renatofilho) wrote :

Are there any related MPs required for this MP to build/function as expected? NO

Is your branch in sync with latest trunk (e.g. bzr pull lp:trunk -> no changes) YES

Did you perform an exploratory manual test run of your code change and any related functionality on device or emulator? YES

Did you successfully run all tests found in your component's Test Plan (https://wiki.ubuntu.com/Process/Merges/TestPlan/address-book-app) on device or emulator? YES

If you changed the UI, was the change specified/approved by design? NO UI CHANGE

If you changed the packaging (debian), did you subscribe a core-dev to this MP? NO PACKAGE CHANGE

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/imports/ContactEdit/ContactDetailWithTypeEditor.qml'
--- src/imports/ContactEdit/ContactDetailWithTypeEditor.qml 2013-11-22 01:05:52 +0000
+++ src/imports/ContactEdit/ContactDetailWithTypeEditor.qml 2014-02-06 22:32:59 +0000
@@ -71,13 +71,14 @@
71 focus: true71 focus: true
72 // disable listview mouse area72 // disable listview mouse area
73 __mouseArea.visible: false73 __mouseArea.visible: false
7474 enabled: root.detail ? !root.detail.readOnly : false
75 implicitHeight: detailTypeSelector.height + fieldValues.height + units.gu(2)75 implicitHeight: detailTypeSelector.height + fieldValues.height + units.gu(2)
7676 opacity: enabled ? 1.0 : 0.5
77 ValueSelector {77 ValueSelector {
78 id: detailTypeSelector78 id: detailTypeSelector
79 objectName: detail ? "type_" + detailToString(detail.type, -1) + "_" + index : ""79 objectName: detail ? "type_" + detailToString(detail.type, -1) + "_" + index : ""
8080
81 readOnly: root.detail ? root.detail.readOnly : false
81 visible: (currentIndex != -1)82 visible: (currentIndex != -1)
82 active: root.active83 active: root.active
83 anchors {84 anchors {
8485
=== modified file 'src/imports/ContactEdit/TextInputDetail.qml'
--- src/imports/ContactEdit/TextInputDetail.qml 2013-10-08 12:58:44 +0000
+++ src/imports/ContactEdit/TextInputDetail.qml 2014-02-06 22:32:59 +0000
@@ -31,6 +31,7 @@
3131
32 Component.onCompleted: makeMeVisible(root)32 Component.onCompleted: makeMeVisible(root)
3333
34 readOnly: detail ? detail.readOnly : true
34 focus: true35 focus: true
35 text: originalValue ? originalValue : ""36 text: originalValue ? originalValue : ""
36 style: TextFieldStyle {37 style: TextFieldStyle {
3738
=== modified file 'src/imports/ContactEdit/ValueSelector.qml'
--- src/imports/ContactEdit/ValueSelector.qml 2013-11-22 00:47:10 +0000
+++ src/imports/ContactEdit/ValueSelector.qml 2014-02-06 22:32:59 +0000
@@ -20,6 +20,7 @@
20Item {20Item {
21 id: root21 id: root
2222
23 property bool readOnly: false
23 property bool active: false24 property bool active: false
24 property alias values: listView.model25 property alias values: listView.model
25 property alias currentIndex: listView.currentIndex26 property alias currentIndex: listView.currentIndex
@@ -120,7 +121,7 @@
120 MouseArea {121 MouseArea {
121 anchors.fill: parent122 anchors.fill: parent
122 propagateComposedEvents: true123 propagateComposedEvents: true
123 onClicked: root.state = "expanded"124 onClicked: if (!readOnly) root.state = "expanded"
124 }125 }
125126
126 ListView {127 ListView {

Subscribers

People subscribed via source and target branches