Merge lp:~cimi/unity8/fix-1236286 into lp:unity8

Proposed by Andrea Cimitan
Status: Merged
Approved by: Michał Sawicz
Approved revision: 481
Merged at revision: 495
Proposed branch: lp:~cimi/unity8/fix-1236286
Merge into: lp:unity8
Diff against target: 82 lines (+15/-10)
2 files modified
Launcher/Launcher.qml (+1/-1)
tests/qmltests/tst_Shell.qml (+14/-9)
To merge this branch: bzr merge lp:~cimi/unity8/fix-1236286
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Michael Zanetti (community) Approve
Review via email: mp+192399@code.launchpad.net

Commit message

Threshold to activate the left edge swipe to reveal dash is now 26 GU

Description of the change

Threshold to activate the left edge swipe to reveal dash is now 26 GU

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:479
http://jenkins.qa.ubuntu.com/job/unity8-ci/1485/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/38
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/38/console
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/9
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-amd64-ci/9
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/9
        deb: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/9/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-i386-ci/9
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/32
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/38
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/38/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/38
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/38/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/2667
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/2718/console
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/373
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/374

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/unity8-ci/1485/rebuild

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

please adjust the left-edge qml test to check for the distance. For example by once releasing at 25 gus and checking if it's NOT going to the dash, and once with 27 gus and check if it does.

review: Needs Fixing
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:480
http://jenkins.qa.ubuntu.com/job/unity8-ci/1491/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/77/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/76
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/15
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-amd64-ci/15
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/15
        deb: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/15/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-i386-ci/15
    FAILURE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/68/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/77
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/77/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/76
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/76/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/2703
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/2754
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/449
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/438

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/unity8-ci/1491/rebuild

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

thanks :)

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Autolanding.
Unapproved changes made after approval.
http://jenkins.qa.ubuntu.com/job/unity8-autolanding/638/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/388
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/376
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/111
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-amd64-autolanding/24
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-autolanding/24
        deb: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-autolanding/24/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-i386-autolanding/24
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/363
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/388
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/388/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/376
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/376/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/2959
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/3043
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/998
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/996

review: Needs Fixing (continuous-integration)
Revision history for this message
Michał Sawicz (saviq) wrote :

Grr ;)

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:481
http://jenkins.qa.ubuntu.com/job/unity8-ci/1561/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/392
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/380
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/114
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-amd64-ci/85
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/85
        deb: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/85/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-i386-ci/85
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/367
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/392
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/392/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/380
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/380/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/2961
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/3046
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/1003
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/1002

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/unity8-ci/1561/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Autolanding.
More details in the following jenkins job:
http://jenkins.qa.ubuntu.com/job/unity8-autolanding/639/
Executed test runs:
    SUCCESS: http://10.97.0.26:8080/job/generic-cleanup-mbs/3087
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/393
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/381
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/115
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-amd64-autolanding/25
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-autolanding/25
        deb: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-autolanding/25/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-i386-autolanding/25
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/368
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/393
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/393/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/381
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/381/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/3049
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/1007

review: Needs Fixing (continuous-integration)
Revision history for this message
Michał Sawicz (saviq) wrote :

Broken maguros, had to restart.

Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'Launcher/Launcher.qml'
2--- Launcher/Launcher.qml 2013-09-12 19:08:06 +0000
3+++ Launcher/Launcher.qml 2013-10-25 13:53:41 +0000
4@@ -28,7 +28,7 @@
5
6 property int panelWidth: units.gu(8)
7 property int dragAreaWidth: units.gu(1)
8- property int minimizeDistance: panelWidth * 2.5
9+ property int minimizeDistance: units.gu(26)
10 property real progress: dragArea.dragging && dragArea.touchX > panelWidth ?
11 (width * (dragArea.touchX-panelWidth) / (width - panelWidth)) : 0
12
13
14=== modified file 'tests/qmltests/tst_Shell.qml'
15--- tests/qmltests/tst_Shell.qml 2013-10-09 20:01:12 +0000
16+++ tests/qmltests/tst_Shell.qml 2013-10-25 13:53:41 +0000
17@@ -111,14 +111,14 @@
18 waitUntilApplicationWindowIsFullyVisible();
19
20 // Minimize the application we just launched
21- swipeFromLeftEdge();
22+ swipeFromLeftEdge(units.gu(27));
23
24 waitForUIToSettle();
25
26 checkRightEdgeDragWithMinimizedApp();
27
28 // Minimize that application once again
29- swipeFromLeftEdge();
30+ swipeFromLeftEdge(units.gu(27));
31
32 // Right edge behavior should now be the same as before that app,
33 // was launched. Manually cleanup beforehand to get to initial
34@@ -130,8 +130,10 @@
35
36 function test_leftEdgeDrag_data() {
37 return [
38- {tag: "without launcher", revealLauncher: false},
39- {tag: "with launcher", revealLauncher: true},
40+ {tag: "without launcher", revealLauncher: false, swipeLength: units.gu(27), appHides: true},
41+ {tag: "with launcher", revealLauncher: true, swipeLength: units.gu(27), appHides: true},
42+ {tag: "small swipe", revealLauncher: false, swipeLength: units.gu(25), appHides: false},
43+ {tag: "long swipe", revealLauncher: false, swipeLength: units.gu(27), appHides: true}
44 ];
45 }
46
47@@ -144,8 +146,11 @@
48 dragLauncherIntoView();
49 }
50
51- swipeFromLeftEdge();
52- waitUntilApplicationWindowIsFullyHidden();
53+ swipeFromLeftEdge(data.swipeLength);
54+ if (data.appHides)
55+ waitUntilApplicationWindowIsFullyHidden();
56+ else
57+ waitUntilApplicationWindowIsFullyVisible();
58 }
59
60 function test_suspend() {
61@@ -293,7 +298,7 @@
62 waitUntilApplicationWindowIsFullyVisible();
63
64 // Minimize the application we just launched
65- swipeFromLeftEdge();
66+ swipeFromLeftEdge(units.gu(27));
67
68 // Wait for the whole UI to settle down
69 waitUntilApplicationWindowIsFullyHidden();
70@@ -376,10 +381,10 @@
71 }
72 }
73
74- function swipeFromLeftEdge() {
75+ function swipeFromLeftEdge(swipeLength) {
76 var touchStartX = 2;
77 var touchStartY = shell.height / 2;
78- touchFlick(shell, touchStartX, touchStartY, shell.width * 0.75, touchStartY);
79+ touchFlick(shell, touchStartX, touchStartY, swipeLength, touchStartY);
80 }
81
82 function swipeLeftFromCenter() {

Subscribers

People subscribed via source and target branches