Merge lp:~renatofilho/ubuntu-calendar-app/fix-url-arg into lp:ubuntu-calendar-app

Proposed by Renato Araujo Oliveira Filho
Status: Merged
Approved by: Arthur Mello
Approved revision: 796
Merged at revision: 799
Proposed branch: lp:~renatofilho/ubuntu-calendar-app/fix-url-arg
Merge into: lp:ubuntu-calendar-app
Diff against target: 31 lines (+9/-5)
1 file modified
calendar.qml (+9/-5)
To merge this branch: bzr merge lp:~renatofilho/ubuntu-calendar-app/fix-url-arg
Reviewer Review Type Date Requested Status
Arthur Mello (community) Approve
Jenkins Bot continuous-integration Approve
Review via email: mp+289885@code.launchpad.net

Commit message

Show correct date and time when url handler is invoked with the app open.

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
Arthur Mello (artmello) wrote :

lgtm

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'calendar.qml'
2--- calendar.qml 2016-03-22 20:28:17 +0000
3+++ calendar.qml 2016-03-23 12:00:03 +0000
4@@ -126,7 +126,11 @@
5 handleUri(uris[0])
6 if (tabs.starttime !== -1) {
7 tabs.currentDay = new Date(tabs.starttime);
8- tabs.selectedTabIndex = dayTab.index
9+ if (tabs.selectedTabIndex != dayTab.index)
10+ tabs.selectedTabIndex = dayTab.index
11+ else {
12+ dayTab.page.item.showDate(tabs.currentDay)
13+ }
14 tabs.starttime = -1
15 }
16 }
17@@ -735,10 +739,10 @@
18 function showDate(date)
19 {
20 var dateGoTo = new Date(date)
21- if (!anchorDate ||
22- (dateGoTo.getFullYear() != anchorDate.getFullYear()) ||
23- (dateGoTo.getMonth() != anchorDate.getMonth()) ||
24- (dateGoTo.getDate() != anchorDate.getDate())) {
25+ if (!currentDate ||
26+ (dateGoTo.getFullYear() !== currentDate.getFullYear()) ||
27+ (dateGoTo.getMonth() !== currentDate.getMonth()) ||
28+ (dateGoTo.getDate() !== currentDate.getDate())) {
29 anchorDate = new Date(dateGoTo)
30 }
31 delayScrollToDate(dateGoTo)

Subscribers

People subscribed via source and target branches

to status/vote changes: