Merge lp:~renatofilho/ubuntu-calendar-app/destroy-bottom-edge-page into lp:ubuntu-calendar-app

[Renato Araujo Oliveira Filho]

Make sure that bottom edge page is destroyed after collapse it.

[Jenkins Bot (ubuntu-core-apps-jenkins-bot)]

PASSED: Continuous integration, rev:821
https://core-apps-jenkins.ubuntu.com/job/calendar-app-ci/828/
Executed test runs:
    None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/863/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/calendar-app-ci/828/rebuild

review: Approve (continuous-integration)

[Arthur Mello (artmello)]

lgtm

review: Approve

=== modified file 'NewEventBottomEdge.qml'
--- NewEventBottomEdge.qml	2016-03-18 00:40:37 +0000
+++ NewEventBottomEdge.qml	2016-04-13 21:24:53 +0000
@@ -63,13 +63,6 @@
         implicitWidth: bottomEdge.width
         implicitHeight: bottomEdge.height
         children: bottomEdge._realPage
-        Component.onDestruction: {
-            if (bottomEdge._realPage) {
-                bottomEdge._realPage.destroy()
-                bottomEdge._realPage = null
-                _realPage = editorPageBottomEdge.createObject(null)
-            }
-        }
     }
 
     onCommitStarted: {
@@ -77,6 +70,14 @@
         updateNewEventInfo(bottomEdge.date ? bottomEdge.date : new Date(), false)
     }
 
+    onCollapseCompleted: {
+        if (bottomEdge._realPage) {
+            bottomEdge._realPage.destroy()
+            bottomEdge._realPage = null
+            _realPage = editorPageBottomEdge.createObject(null)
+        }
+    }
+
     Component.onCompleted:  {
         if (eventModel)
             _realPage = editorPageBottomEdge.createObject(null)