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

Proposed by Albert Astals Cid
Status: Merged
Approved by: Josh Arenson
Approved revision: 2546
Merged at revision: 2617
Proposed branch: lp:~aacid/unity8/touchdownLoader
Merge into: lp:unity8
Diff against target: 466 lines (+151/-106)
17 files modified
plugins/Dash/CardCreator.js (+8/-6)
tests/plugins/Dash/cardcreator/1.res (+8/-6)
tests/plugins/Dash/cardcreator/1.res.cardcreator (+8/-6)
tests/plugins/Dash/cardcreator/11.res (+10/-8)
tests/plugins/Dash/cardcreator/11.res.cardcreator (+10/-8)
tests/plugins/Dash/cardcreator/12.res (+8/-6)
tests/plugins/Dash/cardcreator/12.res.cardcreator (+8/-6)
tests/plugins/Dash/cardcreator/4.res (+8/-6)
tests/plugins/Dash/cardcreator/4.res.cardcreator (+8/-6)
tests/plugins/Dash/cardcreator/6.res (+8/-6)
tests/plugins/Dash/cardcreator/6.res.cardcreator (+8/-6)
tests/plugins/Dash/cardcreator/8.res (+8/-6)
tests/plugins/Dash/cardcreator/8.res.cardcreator (+8/-6)
tests/plugins/Dash/cardcreator/9.res (+10/-8)
tests/plugins/Dash/cardcreator/9.res.cardcreator (+10/-8)
tests/qmltests/Dash/tst_Card.qml (+7/-2)
tests/qmltests/Dash/tst_GenericScopeView.qml (+16/-6)
To merge this branch: bzr merge lp:~aacid/unity8/touchdownLoader
Reviewer Review Type Date Requested Status
Unity8 CI Bot continuous-integration Approve
Josh Arenson Approve
Andrea Cimitan (community) Approve
Michael Terry Approve
Review via email: mp+300735@code.launchpad.net

Commit message

Put the touchdown shape inside a loader

We only create the shape when needed, saving a bit of CPU time when we're just scrolling around the scopes

Doesn't seem to make the touchdown appear time on press noticeably slower

Description of the change

 * 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

 * 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

To post a comment you must log in.
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2543
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1782/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/2336
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1249
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1249
    FAILURE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1249/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2364
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2254
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2254
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2254
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2248
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2248/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2248
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2248/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2248
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2248/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2248
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2248/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2248
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2248/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2248
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2248/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2248
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2248/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2248
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2248/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2248
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2248/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1782/rebuild

review: Needs Fixing (continuous-integration)
lp:~aacid/unity8/touchdownLoader updated
2544. By Albert Astals Cid

Merge unity8

2545. By Albert Astals Cid

Adapt test to code changes

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2543
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1800/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/2354
    FAILURE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1263/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1263/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1263/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2382
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2269
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2269
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2269
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2262
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2262/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2262
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2262/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2262
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2262/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2262
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2262/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2262
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2262/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2262
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2262/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2262
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2262/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2262
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2262/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2262
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2262/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1800/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2545
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1802/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/2356
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1264
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1264
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1264
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2384
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2271
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2271
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2271
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2264
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2264/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2264
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2264/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2264
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2264/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2264
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2264/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2264
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2264/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2264
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2264/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2264
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2264/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2264
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2264/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2264
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2264/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1802/rebuild

review: Needs Fixing (continuous-integration)
lp:~aacid/unity8/touchdownLoader updated
2546. By Albert Astals Cid

Merge

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2546
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1902/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/2478/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2506
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2392
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2392
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2392
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2385
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2385/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2385
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2385/artifact/output/*zip*/output.zip
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2385/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2385
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2385/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2385
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2385/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2385
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2385/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2385
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2385/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2385
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2385/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2385
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2385/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1902/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Michael Terry (mterry) wrote :

LGTM. Tested and it seems just as fast. Thanks!

review: Approve
Revision history for this message
Andrea Cimitan (cimi) wrote :

fine with the change too

review: Approve
Revision history for this message
Josh Arenson (josharenson) wrote :

lgtm as well!

review: Approve
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

PASSED: Continuous integration, rev:2546
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/2022/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/2653
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1454
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1454
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1454
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2681
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2554
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2554
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2554
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2548
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2548/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2548
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2548/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2548
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2548/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2548
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2548/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2548
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2548/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2548
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2548/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2548
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2548/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2548
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2548/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2548
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2548/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/2022/rebuild

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plugins/Dash/CardCreator.js'
--- plugins/Dash/CardCreator.js 2016-08-03 15:49:54 +0000
+++ plugins/Dash/CardCreator.js 2016-08-08 08:57:51 +0000
@@ -331,13 +331,15 @@
331 }\n';331 }\n';
332332
333// %1 is used as anchors of touchdown effect333// %1 is used as anchors of touchdown effect
334var kTouchdownCode = 'UbuntuShape { \n\334var kTouchdownCode = 'Loader { \n\
335 id: touchdown; \n\335 active: root.pressed; \n\
336 objectName: "touchdown"; \n\
337 anchors { %1 } \n\336 anchors { %1 } \n\
338 visible: root.pressed; \n\337 sourceComponent: UbuntuShape { \n\
339 radius: "medium"; \n\338 objectName: "touchdown"; \n\
340 borderSource: "radius_pressed.sci" \n\339 anchors.fill: parent; \n\
340 radius: "medium"; \n\
341 borderSource: "radius_pressed.sci" \n\
342 } \n\
341 }\n';343 }\n';
342344
343// %1 is used as anchors of subtitleLabel345// %1 is used as anchors of subtitleLabel
344346
=== modified file 'tests/plugins/Dash/cardcreator/1.res'
--- tests/plugins/Dash/cardcreator/1.res 2016-06-28 15:38:21 +0000
+++ tests/plugins/Dash/cardcreator/1.res 2016-08-08 08:57:51 +0000
@@ -69,13 +69,15 @@
69 font.weight: cardData && cardData["subtitle"] ? Font.DemiBold : Font.Normal; 69 font.weight: cardData && cardData["subtitle"] ? Font.DemiBold : Font.Normal;
70 horizontalAlignment: Text.AlignHCenter;70 horizontalAlignment: Text.AlignHCenter;
71 }71 }
72UbuntuShape {72Loader {
73 id: touchdown;73 active: root.pressed;
74 objectName: "touchdown";
75 anchors { fill: artShapeLoader }74 anchors { fill: artShapeLoader }
76 visible: root.pressed;75 sourceComponent: UbuntuShape {
77 radius: "medium";76 objectName: "touchdown";
78 borderSource: "radius_pressed.sci"77 anchors.fill: parent;
78 radius: "medium";
79 borderSource: "radius_pressed.sci"
80 }
79}81}
80implicitHeight: titleLabel.y + titleLabel.height + units.gu(1);82implicitHeight: titleLabel.y + titleLabel.height + units.gu(1);
81}83}
8284
=== modified file 'tests/plugins/Dash/cardcreator/1.res.cardcreator'
--- tests/plugins/Dash/cardcreator/1.res.cardcreator 2016-06-28 15:50:44 +0000
+++ tests/plugins/Dash/cardcreator/1.res.cardcreator 2016-08-08 08:57:51 +0000
@@ -65,13 +65,15 @@
65 font.weight: cardData && cardData["subtitle"] ? Font.DemiBold : Font.Normal; 65 font.weight: cardData && cardData["subtitle"] ? Font.DemiBold : Font.Normal;
66 horizontalAlignment: Text.AlignHCenter;66 horizontalAlignment: Text.AlignHCenter;
67 }67 }
68UbuntuShape {68Loader {
69 id: touchdown;69 active: root.pressed;
70 objectName: "touchdown";
71 anchors { fill: artShapeLoader }70 anchors { fill: artShapeLoader }
72 visible: root.pressed;71 sourceComponent: UbuntuShape {
73 radius: "medium";72 objectName: "touchdown";
74 borderSource: "radius_pressed.sci"73 anchors.fill: parent;
74 radius: "medium";
75 borderSource: "radius_pressed.sci"
76 }
75}77}
76implicitHeight: titleLabel.y + titleLabel.height + units.gu(1);78implicitHeight: titleLabel.y + titleLabel.height + units.gu(1);
77}79}
7880
=== modified file 'tests/plugins/Dash/cardcreator/11.res'
--- tests/plugins/Dash/cardcreator/11.res 2016-06-28 15:38:21 +0000
+++ tests/plugins/Dash/cardcreator/11.res 2016-08-08 08:57:51 +0000
@@ -163,13 +163,15 @@
163 163
164 ] 164 ]
165 }165 }
166UbuntuShape { 166Loader {
167 id: touchdown; 167 active: root.pressed;
168 objectName: "touchdown"; 168 anchors { fill: backgroundLoader }
169 anchors { fill: backgroundLoader } 169 sourceComponent: UbuntuShape {
170 visible: root.pressed;170 objectName: "touchdown";
171 radius: "medium"; 171 anchors.fill: parent;
172 borderSource: "radius_pressed.sci" 172 radius: "medium";
173 }173 borderSource: "radius_pressed.sci"
174 }
175}
174implicitHeight: row.y + row.height + units.gu(1);176implicitHeight: row.y + row.height + units.gu(1);
175}177}
176178
=== modified file 'tests/plugins/Dash/cardcreator/11.res.cardcreator'
--- tests/plugins/Dash/cardcreator/11.res.cardcreator 2016-06-28 15:50:44 +0000
+++ tests/plugins/Dash/cardcreator/11.res.cardcreator 2016-08-08 08:57:51 +0000
@@ -156,13 +156,15 @@
156 156
157 ] 157 ]
158 }158 }
159UbuntuShape { 159Loader {
160 id: touchdown; 160 active: root.pressed;
161 objectName: "touchdown"; 161 anchors { fill: backgroundLoader }
162 anchors { fill: backgroundLoader } 162 sourceComponent: UbuntuShape {
163 visible: root.pressed;163 objectName: "touchdown";
164 radius: "medium"; 164 anchors.fill: parent;
165 borderSource: "radius_pressed.sci" 165 radius: "medium";
166 }166 borderSource: "radius_pressed.sci"
167 }
168}
167implicitHeight: row.y + row.height + units.gu(1);169implicitHeight: row.y + row.height + units.gu(1);
168}170}
169171
=== modified file 'tests/plugins/Dash/cardcreator/12.res'
--- tests/plugins/Dash/cardcreator/12.res 2016-06-28 15:38:21 +0000
+++ tests/plugins/Dash/cardcreator/12.res 2016-08-08 08:57:51 +0000
@@ -95,13 +95,15 @@
95 model: cardData && cardData["socialActions"];95 model: cardData && cardData["socialActions"];
96 onClicked: root.action(actionId);96 onClicked: root.action(actionId);
97}97}
98UbuntuShape {98Loader {
99 id: touchdown;99 active: root.pressed;
100 objectName: "touchdown";
101 anchors { fill: artShapeLoader }100 anchors { fill: artShapeLoader }
102 visible: root.pressed;101 sourceComponent: UbuntuShape {
103 radius: "medium";102 objectName: "touchdown";
104 borderSource: "radius_pressed.sci"103 anchors.fill: parent;
104 radius: "medium";
105 borderSource: "radius_pressed.sci"
106 }
105}107}
106implicitHeight: socialActionsRow.y + socialActionsRow.height + units.gu(1);108implicitHeight: socialActionsRow.y + socialActionsRow.height + units.gu(1);
107}109}
108110
=== modified file 'tests/plugins/Dash/cardcreator/12.res.cardcreator'
--- tests/plugins/Dash/cardcreator/12.res.cardcreator 2016-06-28 15:50:44 +0000
+++ tests/plugins/Dash/cardcreator/12.res.cardcreator 2016-08-08 08:57:51 +0000
@@ -91,13 +91,15 @@
91 model: cardData && cardData["socialActions"];91 model: cardData && cardData["socialActions"];
92 onClicked: root.action(actionId);92 onClicked: root.action(actionId);
93}93}
94UbuntuShape {94Loader {
95 id: touchdown;95 active: root.pressed;
96 objectName: "touchdown";
97 anchors { fill: artShapeLoader }96 anchors { fill: artShapeLoader }
98 visible: root.pressed;97 sourceComponent: UbuntuShape {
99 radius: "medium";98 objectName: "touchdown";
100 borderSource: "radius_pressed.sci"99 anchors.fill: parent;
100 radius: "medium";
101 borderSource: "radius_pressed.sci"
102 }
101}103}
102implicitHeight: socialActionsRow.y + socialActionsRow.height + units.gu(1);104implicitHeight: socialActionsRow.y + socialActionsRow.height + units.gu(1);
103}105}
104106
=== modified file 'tests/plugins/Dash/cardcreator/4.res'
--- tests/plugins/Dash/cardcreator/4.res 2016-05-26 14:53:20 +0000
+++ tests/plugins/Dash/cardcreator/4.res 2016-08-08 08:57:51 +0000
@@ -98,13 +98,15 @@
98}98}
99]99]
100}100}
101UbuntuShape {101Loader {
102 id: touchdown;102 active: root.pressed;
103 objectName: "touchdown";
104 anchors { fill: root }103 anchors { fill: root }
105 visible: root.pressed;104 sourceComponent: UbuntuShape {
106 radius: "medium";105 objectName: "touchdown";
107 borderSource: "radius_pressed.sci"106 anchors.fill: parent;
107 radius: "medium";
108 borderSource: "radius_pressed.sci"
109 }
108}110}
109implicitHeight: row.y + row.height + units.gu(1);111implicitHeight: row.y + row.height + units.gu(1);
110}112}
111113
=== modified file 'tests/plugins/Dash/cardcreator/4.res.cardcreator'
--- tests/plugins/Dash/cardcreator/4.res.cardcreator 2016-05-26 14:53:20 +0000
+++ tests/plugins/Dash/cardcreator/4.res.cardcreator 2016-08-08 08:57:51 +0000
@@ -95,13 +95,15 @@
95}95}
96]96]
97}97}
98UbuntuShape {98Loader {
99 id: touchdown;99 active: root.pressed;
100 objectName: "touchdown";
101 anchors { fill: root }100 anchors { fill: root }
102 visible: root.pressed;101 sourceComponent: UbuntuShape {
103 radius: "medium";102 objectName: "touchdown";
104 borderSource: "radius_pressed.sci"103 anchors.fill: parent;
104 radius: "medium";
105 borderSource: "radius_pressed.sci"
106 }
105}107}
106implicitHeight: row.y + row.height + units.gu(1);108implicitHeight: row.y + row.height + units.gu(1);
107}109}
108110
=== modified file 'tests/plugins/Dash/cardcreator/6.res'
--- tests/plugins/Dash/cardcreator/6.res 2016-05-26 14:53:20 +0000
+++ tests/plugins/Dash/cardcreator/6.res 2016-08-08 08:57:51 +0000
@@ -114,13 +114,15 @@
114 114
115 ]115 ]
116 }116 }
117UbuntuShape {117Loader {
118 id: touchdown;118 active: root.pressed;
119 objectName: "touchdown";
120 anchors { fill: backgroundLoader }119 anchors { fill: backgroundLoader }
121 visible: root.pressed;120 sourceComponent: UbuntuShape {
122 radius: "medium";121 objectName: "touchdown";
123 borderSource: "radius_pressed.sci"122 anchors.fill: parent;
123 radius: "medium";
124 borderSource: "radius_pressed.sci"
125 }
124}126}
125implicitHeight: headerTitleContainer.y + headerTitleContainer.height + units.gu(1);127implicitHeight: headerTitleContainer.y + headerTitleContainer.height + units.gu(1);
126}128}
127129
=== modified file 'tests/plugins/Dash/cardcreator/6.res.cardcreator'
--- tests/plugins/Dash/cardcreator/6.res.cardcreator 2016-05-26 14:53:20 +0000
+++ tests/plugins/Dash/cardcreator/6.res.cardcreator 2016-08-08 08:57:51 +0000
@@ -112,13 +112,15 @@
112 112
113 ]113 ]
114 }114 }
115UbuntuShape {115Loader {
116 id: touchdown;116 active: root.pressed;
117 objectName: "touchdown";
118 anchors { fill: backgroundLoader }117 anchors { fill: backgroundLoader }
119 visible: root.pressed;118 sourceComponent: UbuntuShape {
120 radius: "medium";119 objectName: "touchdown";
121 borderSource: "radius_pressed.sci"120 anchors.fill: parent;
121 radius: "medium";
122 borderSource: "radius_pressed.sci"
123 }
122}124}
123implicitHeight: headerTitleContainer.y + headerTitleContainer.height + units.gu(1);125implicitHeight: headerTitleContainer.y + headerTitleContainer.height + units.gu(1);
124}126}
125127
=== modified file 'tests/plugins/Dash/cardcreator/8.res'
--- tests/plugins/Dash/cardcreator/8.res 2016-05-26 14:53:20 +0000
+++ tests/plugins/Dash/cardcreator/8.res 2016-08-08 08:57:51 +0000
@@ -96,13 +96,15 @@
96 96
97 ] 97 ]
98 }98 }
99UbuntuShape { 99Loader {
100 id: touchdown;100 active: root.pressed;
101 objectName: "touchdown";
102 anchors { fill: backgroundLoader }101 anchors { fill: backgroundLoader }
103 visible: root.pressed;102 sourceComponent: UbuntuShape {
104 radius: "medium";103 objectName: "touchdown";
105 borderSource: "radius_pressed.sci"104 anchors.fill: parent;
105 radius: "medium";
106 borderSource: "radius_pressed.sci"
107 }
106}108}
107implicitHeight: row.y + row.height + units.gu(1);109implicitHeight: row.y + row.height + units.gu(1);
108}110}
109111
=== modified file 'tests/plugins/Dash/cardcreator/8.res.cardcreator'
--- tests/plugins/Dash/cardcreator/8.res.cardcreator 2016-05-26 14:53:20 +0000
+++ tests/plugins/Dash/cardcreator/8.res.cardcreator 2016-08-08 08:57:51 +0000
@@ -93,13 +93,15 @@
93 93
94 ] 94 ]
95 }95 }
96UbuntuShape { 96Loader {
97 id: touchdown;97 active: root.pressed;
98 objectName: "touchdown";
99 anchors { fill: backgroundLoader }98 anchors { fill: backgroundLoader }
100 visible: root.pressed;99 sourceComponent: UbuntuShape {
101 radius: "medium";100 objectName: "touchdown";
102 borderSource: "radius_pressed.sci"101 anchors.fill: parent;
102 radius: "medium";
103 borderSource: "radius_pressed.sci"
104 }
103}105}
104implicitHeight: row.y + row.height + units.gu(1);106implicitHeight: row.y + row.height + units.gu(1);
105}107}
106108
=== modified file 'tests/plugins/Dash/cardcreator/9.res'
--- tests/plugins/Dash/cardcreator/9.res 2016-05-20 14:53:04 +0000
+++ tests/plugins/Dash/cardcreator/9.res 2016-08-08 08:57:51 +0000
@@ -107,13 +107,15 @@
107 onPressAndHold: { 107 onPressAndHold: {
108 root.pressAndHold(); 108 root.pressAndHold();
109 } 109 }
110 }UbuntuShape { 110 }Loader {
111 id: touchdown; 111 active: root.pressed;
112 objectName: "touchdown"; 112 anchors { fill: root }
113 anchors { fill: root } 113 sourceComponent: UbuntuShape {
114 visible: root.pressed;114 objectName: "touchdown";
115 radius: "medium"; 115 anchors.fill: parent;
116 borderSource: "radius_pressed.sci" 116 radius: "medium";
117 }117 borderSource: "radius_pressed.sci"
118 }
119}
118implicitHeight: audioButton.height;120implicitHeight: audioButton.height;
119}121}
120122
=== modified file 'tests/plugins/Dash/cardcreator/9.res.cardcreator'
--- tests/plugins/Dash/cardcreator/9.res.cardcreator 2016-05-20 14:53:04 +0000
+++ tests/plugins/Dash/cardcreator/9.res.cardcreator 2016-08-08 08:57:51 +0000
@@ -105,13 +105,15 @@
105 onPressAndHold: { 105 onPressAndHold: {
106 root.pressAndHold(); 106 root.pressAndHold();
107 } 107 }
108 }UbuntuShape { 108 }Loader {
109 id: touchdown; 109 active: root.pressed;
110 objectName: "touchdown"; 110 anchors { fill: root }
111 anchors { fill: root } 111 sourceComponent: UbuntuShape {
112 visible: root.pressed;112 objectName: "touchdown";
113 radius: "medium"; 113 anchors.fill: parent;
114 borderSource: "radius_pressed.sci" 114 radius: "medium";
115 }115 borderSource: "radius_pressed.sci"
116 }
117}
116implicitHeight: audioButton.height;118implicitHeight: audioButton.height;
117}119}
118120
=== modified file 'tests/qmltests/Dash/tst_Card.qml'
--- tests/qmltests/Dash/tst_Card.qml 2016-06-02 08:10:48 +0000
+++ tests/qmltests/Dash/tst_Card.qml 2016-08-08 08:57:51 +0000
@@ -618,12 +618,17 @@
618 waitForRendering(card);618 waitForRendering(card);
619619
620 var touchdown = findChild(card, "touchdown");620 var touchdown = findChild(card, "touchdown");
621 compare(touchdown, null);
621622
622 compare(touchdown.visible, false);
623 mousePress(card);623 mousePress(card);
624
625 touchdown = findChild(card, "touchdown");
624 compare(touchdown.visible, true);626 compare(touchdown.visible, true);
627
625 mouseRelease(card);628 mouseRelease(card);
626 compare(touchdown.visible, false);629
630 touchdown = findChild(card, "touchdown");
631 compare(touchdown, null);
627 }632 }
628633
629 function test_paddings_data() {634 function test_paddings_data() {
630635
=== modified file 'tests/qmltests/Dash/tst_GenericScopeView.qml'
--- tests/qmltests/Dash/tst_GenericScopeView.qml 2016-06-29 14:45:18 +0000
+++ tests/qmltests/Dash/tst_GenericScopeView.qml 2016-08-08 08:57:51 +0000
@@ -642,11 +642,15 @@
642 var item0 = findChild(cardGrid, "delegate0");642 var item0 = findChild(cardGrid, "delegate0");
643 waitForRendering(item0);643 waitForRendering(item0);
644 compare(item0.enabled, false);644 compare(item0.enabled, false);
645
645 var touchdown = findChild(item0, "touchdown");646 var touchdown = findChild(item0, "touchdown");
647 compare(touchdown, null);
646648
647 compare(touchdown.visible, false);
648 mousePress(item0);649 mousePress(item0);
649 compare(touchdown.visible, false);650
651 touchdown = findChild(item0, "touchdown");
652 compare(touchdown, null);
653
650 mouseRelease(item0);654 mouseRelease(item0);
651655
652 cardTool.template["non-interactive"] = false;656 cardTool.template["non-interactive"] = false;
@@ -656,13 +660,19 @@
656 waitForRendering(category0);660 waitForRendering(category0);
657 item0 = findChild(cardGrid, "delegate0");661 item0 = findChild(cardGrid, "delegate0");
658 compare(item0.enabled, true);662 compare(item0.enabled, true);
659 var touchdown = findChild(item0, "touchdown");663
660664 touchdown = findChild(item0, "touchdown");
661 compare(touchdown.visible, false);665 compare(touchdown, null);
666
662 mousePress(item0);667 mousePress(item0);
668
669 touchdown = findChild(item0, "touchdown");
663 compare(touchdown.visible, true);670 compare(touchdown.visible, true);
671
664 mouseRelease(item0);672 mouseRelease(item0);
665 compare(touchdown.visible, false);673
674 touchdown = findChild(item0, "touchdown");
675 compare(touchdown, null);
666 closePreview();676 closePreview();
667 }677 }
668678

Subscribers

People subscribed via source and target branches