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

Proposed by Renato Araujo Oliveira Filho
Status: Merged
Approved by: Bill Filler
Approved revision: 577
Merged at revision: 577
Proposed branch: lp:~renatofilho/address-book-app/fix-header-margin
Merge into: lp:address-book-app
Diff against target: 135 lines (+57/-40)
2 files modified
src/imports/Settings/SettingsPage.qml (+12/-3)
src/imports/Ubuntu/Contacts/SIMCardImportPage.qml (+45/-37)
To merge this branch: bzr merge lp:~renatofilho/address-book-app/fix-header-margin
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Bill Filler (community) Needs Fixing
Review via email: mp+292464@code.launchpad.net

Commit message

Fixed page header.

To post a comment you must log in.
576. By Renato Araujo Oliveira Filho

Fixed page header.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Bill Filler (bfiller) wrote :

Phone number is still obscured by header

review: Needs Fixing
577. By Renato Araujo Oliveira Filho

Use fixed header for Setting page.

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

@bfiller, sorry I discovery that SDK header still buggy in some situations. After a discussion with SDK guys we agree in use fixed headers for now.

Silo is up-to-date now.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
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/Settings/SettingsPage.qml'
--- src/imports/Settings/SettingsPage.qml 2016-03-09 19:23:46 +0000
+++ src/imports/Settings/SettingsPage.qml 2016-04-26 00:49:02 +0000
@@ -27,7 +27,12 @@
2727
28 property var contactListModel28 property var contactListModel
2929
30 title: i18n.tr("Settings")30 header: PageHeader {
31 id: pageHeader
32
33 title: i18n.tr("Settings")
34 flickable: null
35 }
3136
32 ContactsUI.SIMList {37 ContactsUI.SIMList {
33 id: simList38 id: simList
@@ -37,11 +42,15 @@
37 id: myself42 id: myself
38 }43 }
3944
40 flickable: null
41 Flickable {45 Flickable {
42 id: numberFlickable46 id: numberFlickable
43 contentHeight: childrenRect.height47 contentHeight: childrenRect.height
44 anchors.fill: parent48 anchors {
49 top: pageHeader.bottom
50 left: parent.left
51 right: parent.right
52 bottom: parent.bottom
53 }
45 clip: true54 clip: true
4655
47 Column {56 Column {
4857
=== modified file 'src/imports/Ubuntu/Contacts/SIMCardImportPage.qml'
--- src/imports/Ubuntu/Contacts/SIMCardImportPage.qml 2016-02-04 14:15:10 +0000
+++ src/imports/Ubuntu/Contacts/SIMCardImportPage.qml 2016-04-26 00:49:02 +0000
@@ -33,7 +33,50 @@
3333
34 signal importCompleted()34 signal importCompleted()
3535
36 title: i18n.dtr("address-book-app", "SIM contacts")36 header: PageHeader {
37 id: pageHeader
38
39 title: i18n.dtr("address-book-app", "SIM contacts")
40 flickable: contactList.view
41 trailingActionBar {
42 actions: [
43 Action {
44 text: (contactList.selectedItems.count === contactList.count) ?
45 i18n.dtr("address-book-app", "Unselect All") :
46 i18n.dtr("address-book-app", "Select All")
47 iconName: "select"
48 onTriggered: {
49 if (contactList.selectedItems.count === contactList.count) {
50 contactList.clearSelection()
51 } else {
52 contactList.selectAll()
53 }
54 }
55 visible: (contactList.count > 0)
56 },
57 Action {
58 text: i18n.dtr("address-book-app", "Import")
59 objectName: "confirmImport"
60
61 iconName: "tick"
62 enabled: (contactList.selectedItems.count > 0)
63 onTriggered: {
64 root.state = "importing"
65 var contacts = []
66 var items = contactList.selectedItems
67
68 for (var i=0, iMax=items.count; i < iMax; i++) {
69 contacts.push(items.get(i).model.contact)
70 }
71
72 contactList.listModel.exportContacts(root.exportFile,
73 [],
74 contacts)
75 }
76 }
77 ]
78 }
79 }
3780
38 function lockedSIMCount()81 function lockedSIMCount()
39 {82 {
@@ -197,42 +240,7 @@
197 }240 }
198 }241 }
199242
200 head.actions: [243
201 Action {
202 text: (contactList.selectedItems.count === contactList.count) ?
203 i18n.dtr("address-book-app", "Unselect All") :
204 i18n.dtr("address-book-app", "Select All")
205 iconName: "select"
206 onTriggered: {
207 if (contactList.selectedItems.count === contactList.count) {
208 contactList.clearSelection()
209 } else {
210 contactList.selectAll()
211 }
212 }
213 visible: (contactList.count > 0)
214 },
215 Action {
216 text: i18n.dtr("address-book-app", "Import")
217 objectName: "confirmImport"
218
219 iconName: "tick"
220 enabled: (contactList.selectedItems.count > 0)
221 onTriggered: {
222 root.state = "importing"
223 var contacts = []
224 var items = contactList.selectedItems
225
226 for (var i=0, iMax=items.count; i < iMax; i++) {
227 contacts.push(items.get(i).model.contact)
228 }
229
230 contactList.listModel.exportContacts(root.exportFile,
231 [],
232 contacts)
233 }
234 }
235 ]
236244
237 states: [245 states: [
238 State {246 State {

Subscribers

People subscribed via source and target branches