Merge lp:~vthompson/ubuntu-weather-app/reboot-pull-refresh into lp:ubuntu-weather-app

[Victor Thompson]

* Add pull to refresh support

This is a simple change to get "Pull to refresh" working. Currently, the consensus is that we will add an icon to trigger a refresh of the data--however, I wanted to see what other people thought of this approach.

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

PASSED: Continuous integration, rev:19
http://91.189.93.70:8080/job/ubuntu-weather-app-reboot-ci/40/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-weather-app-reboot-utopic-amd64-ci/10
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-weather-app-reboot-vivid-amd64-ci/40

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-weather-app-reboot-ci/40/rebuild

review: Approve (continuous-integration)

[Martin Borho (martin-borho)]

I'm totally fine with this! Haven't thought it's that easy, feels great to use!

Nonetheless, it does not pull back after the refresh has finished. I did this to have it closed after refresh: http://paste.ubuntu.com/10590531/

review: Needs Fixing

[Victor Thompson (vthompson)]

Ah, great! Thanks Martin!

[lp:~vthompson/ubuntu-weather-app/reboot-pull-refresh]

20. By Victor Thompson on 2015-03-13

Fix indicator not going away.

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

PASSED: Continuous integration, rev:20
http://91.189.93.70:8080/job/ubuntu-weather-app-reboot-ci/42/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-weather-app-reboot-utopic-amd64-ci/12
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-weather-app-reboot-vivid-amd64-ci/42

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-weather-app-reboot-ci/42/rebuild

review: Approve (continuous-integration)

[Martin Borho (martin-borho)]

Thanks, LGTM!

review: Approve

=== modified file 'app/ui/HomePage.qml'
--- app/ui/HomePage.qml	2015-03-03 18:37:59 +0000
+++ app/ui/HomePage.qml	2015-03-13 15:47:25 +0000
@@ -87,6 +87,16 @@
         height: parent.height
         contentWidth: parent.width
 
+        PullToRefresh {
+            id: pullToRefresh
+            parent: locationFlickable
+            refreshing: false
+            onRefresh: {
+                refreshing = true
+                refreshData(false, true)
+            }
+        }
+
         /*
           ListView for locations with snap-scrolling horizontally.
         */
@@ -113,6 +123,7 @@
                 currentIndex = settings.current
 
                 if (model > 0) {
+                    pullToRefresh.refreshing = false
                     loaded = true
                 }
             }