Merge lp:~aacid/unity8/vj_reset_cardtool_settle into lp:unity8

[Albert Astals Cid]

Reset VerticalJournal until the cardTool settles

 * Are there any related MPs required for this MP to build/function as expected?
No

 * Did you perform an exploratory manual test run of your code change and any related functionality?
Yes

 * Did you make sure that your branch does not contain spurious tags?
Yes

 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A

 * If you changed the UI, has there been a design review?
N/A

[Michał Sawicz (saviq)]

 * Did you perform an exploratory manual test run of the code change and any related functionality?
Will do in silo.
 * Did CI run pass? If not, please explain why.
Not yet, expecting AP failures though.
 * Did you make sure that the branch does not contain spurious tags?
Yes.

Could use a test probably.

review: Approve (code)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1367
http://jenkins.qa.ubuntu.com/job/unity8-ci/4664/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/5963
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/1650
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/1760
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/1758
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/5605
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/7215
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/7215/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/14724

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/4664/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1367
http://jenkins.qa.ubuntu.com/job/unity8-ci/4687/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/6041
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/1673
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/1783
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/1781
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/5680
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/7293
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/7293/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/14843

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/4687/rebuild

review: Needs Fixing (continuous-integration)

=== modified file 'qml/Dash/CardVerticalJournal.qml'
--- qml/Dash/CardVerticalJournal.qml	2014-09-30 09:23:23 +0000
+++ qml/Dash/CardVerticalJournal.qml	2014-10-15 15:05:49 +0000
@@ -33,6 +33,23 @@
     // in case the model is non empty
     readonly property double minHeight: root.model.count >= 1 ? cardVerticalJournal.rowSpacing + 1 : 0
 
+    function clearView() {
+        cardVerticalJournal.model = null;
+        cardVerticalJournal.model = root.model;
+    }
+
+    // Clear the view if the cardTool changes its sizes
+    // means it is still settling. This is
+    // necessary because the VerticalJournal does
+    // not support its elements changing height so
+    // we need to construct all the items with the settled
+    // size of the cardTool
+    Connections {
+        target: cardTool
+        onArtShapeSizeChanged: root.clearView();
+        onHeaderHeightChanged: root.clearView();
+    }
+
     ResponsiveVerticalJournal {
         id: cardVerticalJournal