Merge lp:~nik90/ubuntu-calendar-app/replace-x-icon into lp:ubuntu-calendar-app

Proposed by Nekhelesh Ramananthan
Status: Merged
Approved by: Nekhelesh Ramananthan
Approved revision: 453
Merged at revision: 452
Proposed branch: lp:~nik90/ubuntu-calendar-app/replace-x-icon
Merge into: lp:ubuntu-calendar-app
Diff against target: 31 lines (+6/-15)
1 file modified
NewEvent.qml (+6/-15)
To merge this branch: bzr merge lp:~nik90/ubuntu-calendar-app/replace-x-icon
Reviewer Review Type Date Requested Status
Mihir Soni Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+234851@code.launchpad.net

Commit message

Replaces the "X" icon in the NewEvent page with the standard "<" icon as requested by design for clock app.

Description of the change

This MP replaces the "X" icon in the NewEvent page with the standard "<" icon as requested by design for clock app. However this should also apply for the calendar app. hence proposing it here.

To post a comment you must log in.
453. By Nekhelesh Ramananthan

Small code refactor

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Mihir Soni (mihirsoni) wrote :

Looks good to me.
Thanks tested and working as expected.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'NewEvent.qml'
--- NewEvent.qml 2014-09-16 14:13:17 +0000
+++ NewEvent.qml 2014-09-16 17:14:22 +0000
@@ -53,21 +53,12 @@
53 endTimeInput.text = Qt.formatTime(endDate);53 endTimeInput.text = Qt.formatTime(endDate);
54 }54 }
5555
56 head {56 head.actions: Action {
57 backAction: Action {57 iconName: "ok"
58 iconName: "close"58 objectName: "save"
59 onTriggered: pageStack.pop();59 text: i18n.tr("Save")
60 }60 enabled: !!titleEdit.text.trim()
6161 onTriggered: saveToQtPim();
62 actions: [
63 Action {
64 iconName: "ok"
65 objectName: "save"
66 text: i18n.tr("Save")
67 enabled: !!titleEdit.text.trim()
68 onTriggered: saveToQtPim();
69 }
70 ]
71 }62 }
7263
73 Component.onCompleted: {64 Component.onCompleted: {

Subscribers

People subscribed via source and target branches

to status/vote changes: