Merge lp:~tiagosh/messaging-app/bugfixing into lp:messaging-app

Proposed by Tiago Salem Herrmann
Status: Merged
Approved by: Gustavo Pichorim Boiko
Approved revision: 27
Merged at revision: 25
Proposed branch: lp:~tiagosh/messaging-app/bugfixing
Merge into: lp:messaging-app
Diff against target: 116 lines (+52/-2)
5 files modified
src/qml/MainPage.qml (+2/-0)
src/qml/Messages.qml (+14/-1)
src/qml/ThreadDelegate.qml (+1/-1)
src/qml/assets/new-contact.svg (+34/-0)
src/qml/messaging-app.qml (+1/-0)
To merge this branch: bzr merge lp:~tiagosh/messaging-app/bugfixing
Reviewer Review Type Date Requested Status
Gustavo Pichorim Boiko (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+183027@code.launchpad.net

Commit message

- use proper icon for unknown contacts
- add "new contact" and call buttons to tabbar for unknown contacts
- do not show "select" buttons on empty views

Description of the change

- use proper icon for unknown contacts
- add "new contact" and call buttons to tabbar for unknown contacts
- do not show "select" buttons on empty views

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Gustavo Pichorim Boiko (boiko) wrote :

Looks good.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/qml/MainPage.qml'
2--- src/qml/MainPage.qml 2013-08-21 22:26:13 +0000
3+++ src/qml/MainPage.qml 2013-08-29 21:02:46 +0000
4@@ -28,6 +28,8 @@
5 tools: threadList.isInSelectionMode ? selectionToolbar : regularToolbar
6 title: i18n.tr("Messages")
7
8+ property alias threadCount: threadList.count
9+
10 function startSelection() {
11 threadList.startSelection()
12 }
13
14=== modified file 'src/qml/Messages.qml'
15--- src/qml/Messages.qml 2013-08-26 17:22:11 +0000
16+++ src/qml/Messages.qml 2013-08-29 21:02:46 +0000
17@@ -52,6 +52,7 @@
18 id: messagesToolbar
19 ToolbarButton {
20 objectName: "selectMessagesButton"
21+ visible: messageList.count !== 0
22 action: Action {
23 iconSource: Qt.resolvedUrl("assets/select.png")
24 text: i18n.tr("Select")
25@@ -59,6 +60,18 @@
26 }
27 }
28 ToolbarButton {
29+ visible: contactWatcher.isUnknown && contactWatcher.phoneNumber !== ""
30+ objectName: "addContactButton"
31+ action: Action {
32+ iconSource: Qt.resolvedUrl("assets/new-contact.svg")
33+ text: i18n.tr("Add contact")
34+ onTriggered: {
35+ applicationUtils.switchToAddressbookApp("create://" + contactWatcher.phoneNumber)
36+ messagesToolbar.opened = false
37+ }
38+ }
39+ }
40+ ToolbarButton {
41 visible: !contactWatcher.isUnknown
42 objectName: "contactProfileButton"
43 action: Action {
44@@ -71,7 +84,7 @@
45 }
46 }
47 ToolbarButton {
48- visible: !contactWatcher.isUnknown
49+ visible: contactWatcher.phoneNumber !== ""
50 objectName: "contactCallButton"
51 action: Action {
52 iconSource: Qt.resolvedUrl("assets/call-start.svg")
53
54=== modified file 'src/qml/ThreadDelegate.qml'
55--- src/qml/ThreadDelegate.qml 2013-08-26 17:22:11 +0000
56+++ src/qml/ThreadDelegate.qml 2013-08-29 21:02:46 +0000
57@@ -52,7 +52,7 @@
58 }
59 return Qt.resolvedUrl("assets/avatar-default.png")
60 }
61- return Qt.resolvedUrl("assets/add.svg")
62+ return Qt.resolvedUrl("assets/new-contact.svg")
63 }
64 }
65 MouseArea {
66
67=== added file 'src/qml/assets/new-contact.svg'
68--- src/qml/assets/new-contact.svg 1970-01-01 00:00:00 +0000
69+++ src/qml/assets/new-contact.svg 2013-08-29 21:02:46 +0000
70@@ -0,0 +1,34 @@
71+<?xml version="1.0" encoding="UTF-8" standalone="no"?>
72+<!-- Created with Inkscape (http://www.inkscape.org/) -->
73+<svg id="svg3376" xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#" xmlns="http://www.w3.org/2000/svg" height="90" width="90" version="1.1" xmlns:cc="http://creativecommons.org/ns#" xmlns:dc="http://purl.org/dc/elements/1.1/">
74+ <defs id="defs3378">
75+ <clipPath id="clipPath3008-8" clipPathUnits="userSpaceOnUse">
76+ <path id="path3010-4" style="color:#000000;fill:#f0f" d="m222.01 818.38v53.968h126v-53.968h-41.618c-2.3377 3.8156-5.017 7.3019-8.2185 10.221-3.2833 2.9932-7.3229 5.8744-13.188 5.8744-5.8649 0-9.9045-2.8812-13.188-5.8744-3.2015-2.9186-5.8808-6.4049-8.2185-10.221h-41.57z"/>
77+ </clipPath>
78+ </defs>
79+ <metadata id="metadata3381">
80+ <rdf:RDF>
81+ <cc:Work rdf:about="">
82+ <dc:format>image/svg+xml</dc:format>
83+ <dc:type rdf:resource="http://purl.org/dc/dcmitype/StillImage"/>
84+ <dc:title/>
85+ </cc:Work>
86+ </rdf:RDF>
87+ </metadata>
88+ <g id="layer1" transform="translate(0 -962.36)">
89+ <g id="g4142" transform="translate(-480,48)">
90+ <rect id="rect4006" style="color:#000000;fill:none" height="90" width="90" y="914.36" x="480"/>
91+ <rect id="rect4012" style="color:#000000;fill:#808080" height="42" width="5.9899" y="962.36" x="543.01"/>
92+ <rect id="rect4014" style="color:#000000;fill:#808080" transform="rotate(90)" height="42" width="6" y="-567" x="980.36"/>
93+ <path id="rect3461-3-1" style="color:#000000;fill:#808080" transform="translate(480 914.36)" d="m20.281 36.625c-11.266 0.886-14.281 4.975-14.281 13.563v3.937c0 5.889-0.0044 5.875 7.844 5.875h35.312c7.848 0 7.844 0.014 7.844-5.875v-3.937c0-8.626-3.019-12.701-14.406-13.563-0.768 0.921-1.59 1.792-2.469 2.594-2.147 1.958-4.789 3.843-8.625 3.843s-6.478-1.885-8.625-3.843c-0.874-0.798-1.674-1.678-2.437-2.594-0.05 0.004-0.107-0.004-0.157 0z"/>
94+ <path id="path3447-0-6" style="color:#000000;fill:#808080" d="m511.5 914.36c-9.6428-0.00002-13.5 3.8948-13.5 13.632v1.9474c0 7.7895 7.7369 21.421 13.511 21.421 5.7744 0 13.489-13.632 13.489-21.421v-1.9474c0-9.7368-3.8572-13.632-13.5-13.632z"/>
95+ </g>
96+ </g>
97+ <g id="layer2" style="display:none">
98+ <rect id="rect4006-1" style="color:#000000;display:inline;fill:none" height="90" width="90" y=".0000015822" x="0"/>
99+ <rect id="rect4012-2" style="color:#000000;display:inline;fill:#808080" height="42" width="5.9899" y="48" x="63.01"/>
100+ <rect id="rect4014-9" style="color:#000000;display:inline;fill:#808080" transform="rotate(90)" height="42" width="6" y="-87" x="66"/>
101+ <path id="rect3461-3-1-0" style="color:#000000;display:inline;fill:#808080" d="m246 851.36c0 9 0 9 12 9h54c12 0 12 0 12-9v-6c0-15-6-21-30-21h-18c-24 0-30 6-30 21z" clip-path="url(#clipPath3008-8)" transform="matrix(.65401 0 0 .65432 -154.89 -502.95)"/>
102+ <path id="path3447-0-6-2" style="color:#000000;display:inline;fill:#808080" d="m31.5 0c-9.643-0.00002-13.5 3.8948-13.5 13.632v1.9474c0 7.789 7.737 21.421 13.511 21.421 5.775 0 13.489-13.632 13.489-21.421v-1.9474c0-9.7372-3.857-13.632-13.5-13.632z"/>
103+ </g>
104+</svg>
105
106=== modified file 'src/qml/messaging-app.qml'
107--- src/qml/messaging-app.qml 2013-08-21 23:42:44 +0000
108+++ src/qml/messaging-app.qml 2013-08-29 21:02:46 +0000
109@@ -83,6 +83,7 @@
110 ToolbarItems {
111 id: regularToolbar
112 ToolbarButton {
113+ visible: mainStack.currentPage.threadCount !== 0
114 objectName: "selectButton"
115 text: i18n.tr("Select")
116 iconSource: Qt.resolvedUrl("assets/select.png")

Subscribers

People subscribed via source and target branches