Merge lp:~renatofilho/ubuntu-calendar-app/fix-event-creation into lp:ubuntu-calendar-app

Proposed by Renato Araujo Oliveira Filho
Status: Merged
Approved by: Nekhelesh Ramananthan
Approved revision: 772
Merged at revision: 772
Proposed branch: lp:~renatofilho/ubuntu-calendar-app/fix-event-creation
Merge into: lp:ubuntu-calendar-app
Diff against target: 89 lines (+24/-8)
4 files modified
EventRepetition.qml (+1/-1)
NewEvent.qml (+21/-6)
NewEventBottomEdge.qml (+1/-0)
PageWithBottomEdge.qml (+1/-1)
To merge this branch: bzr merge lp:~renatofilho/ubuntu-calendar-app/fix-event-creation
Reviewer Review Type Date Requested Status
Nekhelesh Ramananthan Approve
Jenkins Bot continuous-integration Approve
Review via email: mp+288326@code.launchpad.net

Commit message

Fixed new event bottom edge page to allow choose repeater and reminders.

To post a comment you must log in.
Revision history for this message
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

LGTM! Thanx for the quick fix. Works as expected.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'EventRepetition.qml'
2--- EventRepetition.qml 2016-01-29 14:47:31 +0000
3+++ EventRepetition.qml 2016-03-07 17:59:02 +0000
4@@ -207,7 +207,7 @@
5 (checked) ? weekDays.push(index) : weekDays.splice(weekDays.indexOf(index),1);
6 }
7 checked:{
8- (weekDays.length === 0 && index === eventRoot.date.getDay() && isEdit === false) ? true : false;
9+ (weekDays.length === 0 && eventRoot.date && (index === eventRoot.date.getDay()) && !isEdit) ? true : false;
10 }
11
12 }
13
14=== modified file 'NewEvent.qml'
15--- NewEvent.qml 2016-03-03 21:42:08 +0000
16+++ NewEvent.qml 2016-03-07 17:59:02 +0000
17@@ -30,6 +30,9 @@
18 id: root
19 objectName: 'newEventPage'
20
21+ // WORKAROUND: allow us to push pages over bottom edge element
22+ property var bottomEdgePageStack: null
23+
24 property var date;
25 property alias allDay: allDayEventCheckbox.checked
26
27@@ -684,7 +687,13 @@
28 visible: (event != undefined) && (event.itemType === Type.Event)
29 text: i18n.tr("Repeats")
30 subText: (event != undefined) && (event.itemType === Type.Event) ? rule === null ? Defines.recurrenceLabel[0] : eventUtils.getRecurrenceString(rule) : ""
31- onClicked: pageStack.push(Qt.resolvedUrl("EventRepetition.qml"),{"eventRoot": root,"isEdit":isEdit});
32+ onClicked: {
33+ var stack = pageStack
34+ if (!stack)
35+ stack = bottomEdgePageStack
36+
37+ stack.push(Qt.resolvedUrl("EventRepetition.qml"),{"eventRoot": root,"isEdit":isEdit});
38+ }
39 }
40
41 ListItems.ThinDivider {
42@@ -717,11 +726,17 @@
43
44 }
45
46- onClicked: pageStack.push(Qt.resolvedUrl("EventReminder.qml"),
47- {"visualReminder": visualReminder,
48- "audibleReminder": audibleReminder,
49- "reminderModel": reminderModel,
50- "eventTitle": titleEdit.text})
51+ onClicked:{
52+ var stack = pageStack
53+ if (!stack)
54+ stack = bottomEdgePageStack
55+
56+ stack.push(Qt.resolvedUrl("EventReminder.qml"),
57+ {"visualReminder": visualReminder,
58+ "audibleReminder": audibleReminder,
59+ "reminderModel": reminderModel,
60+ "eventTitle": titleEdit.text})
61+ }
62 }
63
64 ListItems.ThinDivider {}
65
66=== modified file 'NewEventBottomEdge.qml'
67--- NewEventBottomEdge.qml 2016-02-25 19:36:28 +0000
68+++ NewEventBottomEdge.qml 2016-03-07 17:59:02 +0000
69@@ -99,6 +99,7 @@
70 active: bottomEdge.status === BottomEdge.Committed
71 visible: (bottomEdge.status !== BottomEdge.Hidden)
72 onCanceled: bottomEdge.collapse()
73+ bottomEdgePageStack: bottomEdge.pageStack
74 onEventAdded: {
75 bottomEdge.collapse()
76 bottomEdge.eventCreated(event)
77
78=== modified file 'PageWithBottomEdge.qml'
79--- PageWithBottomEdge.qml 2016-02-25 19:36:28 +0000
80+++ PageWithBottomEdge.qml 2016-03-07 17:59:02 +0000
81@@ -37,7 +37,7 @@
82 NewEventBottomEdge {
83 id: bottomEdge
84
85- pageStack: tabs
86+ pageStack: root.pageStack
87 onOpened: root.bottomEdgeCommitStarted()
88 onEventCreated: root.eventCreated(event)
89 }

Subscribers

People subscribed via source and target branches

to status/vote changes: