Merge lp:~pkunal-parmar/ubuntu-calendar-app/HideTime_AllDayEvent into lp:ubuntu-calendar-app

[Kunal Parmar]

Hiding time in case of Allday event

Hiding time in case of Allday event

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

PASSED: Continuous integration, rev:275
http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/467/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/generic-mediumtests-trusty/2695
        deb: http://91.189.93.70:8080/job/generic-mediumtests-trusty/2695/artifact/work/output/*zip*/output.zip
    SUCCESS: http://91.189.93.70:8080/job/generic-mediumtests-utopic/232
        deb: http://91.189.93.70:8080/job/generic-mediumtests-utopic/232/artifact/work/output/*zip*/output.zip
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-calendar-app-trusty-amd64-ci/301
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-calendar-app-utopic-amd64-ci/25

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-calendar-app-ci/467/rebuild

review: Approve (continuous-integration)

[Mihir Soni (mihirsoni)]

Looks good to me Kunal!!

review: Approve

=== modified file 'EventDetails.qml'
--- EventDetails.qml	2014-05-15 06:39:54 +0000
+++ EventDetails.qml	2014-05-19 15:10:22 +0000
@@ -114,11 +114,14 @@
             requestId = model.fetchItems([e.parentId]);
         }
 
+        allDayEventCheckbox.checked = e.allDay;
+
+        startHeader.visible = !e.allDay;
+        endHeader.visible = !e.allDay;
+
         startHeader.value = startTime;
         endHeader.value = endTime;
 
-        allDayEventCheckbox.checked = e.allDay;
-
         // This is the event title
         if( e.displayLabel) {
             titleLabel.text = e.displayLabel;