Merge lp:~cimi/unity8/dash-visual-tweaks_pageHeader-bottomEdge into lp:unity8

Proposed by Andrea Cimitan
Status: Merged
Approved by: Albert Astals Cid
Approved revision: 2610
Merged at revision: 2675
Proposed branch: lp:~cimi/unity8/dash-visual-tweaks_pageHeader-bottomEdge
Merge into: lp:unity8
Diff against target: 197 lines (+30/-70)
6 files modified
qml/Dash/Dash.qml (+1/-1)
qml/Dash/DashBackground.qml (+2/-11)
qml/Dash/DashPageHeader.qml (+23/-55)
qml/Dash/GenericScopeView.qml (+1/-0)
tests/qmltests/Dash/tst_DashPageHeader.qml (+2/-2)
tests/qmltests/Dash/tst_GenericScopeView.qml (+1/-1)
To merge this branch: bzr merge lp:~cimi/unity8/dash-visual-tweaks_pageHeader-bottomEdge
Reviewer Review Type Date Requested Status
Unity8 CI Bot continuous-integration Approve
Albert Astals Cid (community) Approve
Review via email: mp+304798@code.launchpad.net

Commit message

Remove background under header pagination dots, update dots assets, remove dash paper background, update bottom edge asset

Description of the change

 * Are there any related MPs required for this MP to build/function as expected? Please list.
n
 * Did you perform an exploratory manual test run of your code change and any related functionality?
y
 * 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?
yes

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

FAILED: Continuous integration, rev:2605
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~cimi/unity8/dash-visual-tweaks_pageHeader-bottomEdge/+merge/304798/+edit-commit-message

https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/2094/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/2742/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2770
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2643
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2643
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2643
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2635
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2635/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2635
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2635/artifact/output/*zip*/output.zip
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2635/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2635
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2635/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2635
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2635/artifact/output/*zip*/output.zip
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2635/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2635
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2635/artifact/output/*zip*/output.zip
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2635/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2635/console

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

review: Needs Fixing (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

You removed the overview_hint file but it's still referenced in Dash.qml
  source: "graphics/overview_hint.png"

review: Needs Fixing
Revision history for this message
Albert Astals Cid (aacid) wrote :

testDashPageHeader doesn't pass

review: Needs Fixing
Revision history for this message
Albert Astals Cid (aacid) wrote :

The pagination dots are not visible if the scope defines a blue background
http://i.imgur.com/FUPj4Uw.png

Is this something we want to live with or design has a solution for it?

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

FAILED: Continuous integration, rev:2607
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/2290/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/3013
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1670
    FAILURE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1670/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1670
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/3041
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2898
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2898/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2898
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2898/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2898
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2898/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2898
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2898/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2898
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2898/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2898
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2898/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2898
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2898/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2898
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2898/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2898
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2898/artifact/output/*zip*/output.zip

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

review: Needs Fixing (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

I think the old Qt doesn't like the alpha components in #00000000 and that makes the test not pass, remove two zeros and it shoudl work i guesS?

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

> The pagination dots are not visible if the scope defines a blue background
> http://i.imgur.com/FUPj4Uw.png
>
> Is this something we want to live with or design has a solution for it?

asked design, we can live with it for now

Revision history for this message
Albert Astals Cid (aacid) wrote :

Ok, i guess that's ok if that's what they want :)

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

 * Did CI run pass? If not, please explain why.
Waiting before top approval

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

PASSED: Continuous integration, rev:2607
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/2296/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/3029
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1680
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1680
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1680
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/3057
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2914
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2914/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2914
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2914/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2914
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2914/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2914
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2914/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2914
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2914/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2914
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2914/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2914
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2914/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2914
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2914/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2914
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2914/artifact/output/*zip*/output.zip

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

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

PASSED: Continuous integration, rev:2608
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/2336/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/3074
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1720
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1720
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1720
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/3102
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2959
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2959/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2959
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2959/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2959
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2959/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2959
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2959/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2959
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2959/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2959
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2959/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2959
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2959/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2959
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2959/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2959
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2959/artifact/output/*zip*/output.zip

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

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

> Ok, i guess that's ok if that's what they want :)
>
> * Did you perform an exploratory manual test run of the code change and any
> related functionality?
> Yes
>
> * Did CI run pass? If not, please explain why.
> Waiting before top approval

last revision contains visual tweaks following design

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

FAILED: Continuous integration, rev:2609
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/2372/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/3125
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1770
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1770
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1770
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/3153
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/3009
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/3009/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3009
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3009/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/3009
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/3009/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/3009
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/3009/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3009
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3009/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/3009
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/3009/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/3009
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/3009/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3009
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3009/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/3009
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/3009/artifact/output/*zip*/output.zip

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

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

FAILED: Continuous integration, rev:2609
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/2373/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/3126
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1771
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1771
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1771
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/3154
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/3010
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/3010/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3010
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3010/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/3010
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/3010/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/3010
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/3010/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3010
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3010/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/3010
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/3010/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/3010
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/3010/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3010
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3010/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/3010
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/3010/artifact/output/*zip*/output.zip

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

review: Needs Fixing (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

Seems you actually broke something in the tests.

review: Needs Fixing
2610. By Andrea Cimitan

try to fix test

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

PASSED: Continuous integration, rev:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/2376/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/3129
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1773
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1773
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1773
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/3157
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/3013
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/3013/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3013
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3013/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/3013
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/3013/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/3013
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/3013/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3013
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3013/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/3013
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/3013/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/3013
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/3013/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3013
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3013/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/3013
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/3013/artifact/output/*zip*/output.zip

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

review: Approve (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) :
review: Approve
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

PASSED: Continuous integration, rev:2610
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/2377/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/3130
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1774
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1774
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1774
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/3158
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/3014
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/3014/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3014
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/3014/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/3014
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/3014/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/3014
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/3014/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3014
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/3014/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/3014
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/3014/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/3014
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/3014/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3014
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/3014/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/3014
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/3014/artifact/output/*zip*/output.zip

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

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'qml/Dash/Dash.qml'
--- qml/Dash/Dash.qml 2016-05-02 07:57:07 +0000
+++ qml/Dash/Dash.qml 2016-10-14 09:08:35 +0000
@@ -329,7 +329,7 @@
329 enabled: bottomEdgeController.progress == 0329 enabled: bottomEdgeController.progress == 0
330 UbuntuNumberAnimation {}330 UbuntuNumberAnimation {}
331 }331 }
332 y: parent.height - height * (1 - bottomEdgeController.progress * 4)332 y: parent.height - height * (1 - bottomEdgeController.progress * 4) - units.gu(1)
333 MouseArea {333 MouseArea {
334 // Eat direct presses on the overview hint so that they do not end up in the card below334 // Eat direct presses on the overview hint so that they do not end up in the card below
335 anchors.fill: parent335 anchors.fill: parent
336336
=== modified file 'qml/Dash/DashBackground.qml'
--- qml/Dash/DashBackground.qml 2016-08-30 14:06:47 +0000
+++ qml/Dash/DashBackground.qml 2016-10-14 09:08:35 +0000
@@ -15,16 +15,7 @@
15 */15 */
1616
17import QtQuick 2.417import QtQuick 2.4
18import QtQuick.Window 2.2
19import ImageCache 0.1
2018
21Image {19Rectangle {
22 source: "image://unity8imagecache/" + Qt.resolvedUrl("graphics/paper.png")20 color: "white"
23 fillMode: Image.PreserveAspectCrop
24 horizontalAlignment: Image.AlignRight
25 verticalAlignment: Image.AlignTop
26 // avoid CPU scaling when window size changes
27 readonly property int maxSize: Math.max(Screen.width, Screen.height)
28 sourceSize.width: maxSize
29 sourceSize.height: 0
30}21}
3122
=== modified file 'qml/Dash/DashPageHeader.qml'
--- qml/Dash/DashPageHeader.qml 2016-08-16 14:06:26 +0000
+++ qml/Dash/DashPageHeader.qml 2016-10-14 09:08:35 +0000
@@ -25,7 +25,7 @@
25 id: root25 id: root
26 objectName: "pageHeader"26 objectName: "pageHeader"
27 implicitHeight: headerContainer.height + signatureLineHeight27 implicitHeight: headerContainer.height + signatureLineHeight
28 readonly property real signatureLineHeight: showSignatureLine ? units.gu(2) : 028 readonly property real signatureLineHeight: showSignatureLine ? units.gu(2.5) : headerBottomLine.height
29 readonly property real headerDividerLuminance: Style.luminance(bottomBorder.color)29 readonly property real headerDividerLuminance: Style.luminance(bottomBorder.color)
3030
31 property int activeFiltersCount: 031 property int activeFiltersCount: 0
@@ -44,7 +44,7 @@
44 property ListModel searchHistory44 property ListModel searchHistory
45 property alias searchQuery: searchTextField.text45 property alias searchQuery: searchTextField.text
46 property alias searchHint: searchTextField.placeholderText46 property alias searchHint: searchTextField.placeholderText
47 property bool showSignatureLine: true47 property bool showSignatureLine: false
4848
49 property int paginationCount: 049 property int paginationCount: 0
50 property int paginationIndex: -150 property int paginationIndex: -1
@@ -414,68 +414,36 @@
414 }414 }
415415
416 Rectangle {416 Rectangle {
417 id: bottomBorder417 id: headerBottomLine
418 anchors {
419 top: headerContainer.bottom
420 left: parent.left
421 right: parent.right
422 }
423 height: units.dp(1)
424 color: theme.palette.normal.base
425 }
426
427 Row {
428 anchors {
429 top: headerContainer.bottom
430 horizontalCenter: headerContainer.horizontalCenter
431 topMargin: units.gu(1)
432 }
418 visible: showSignatureLine433 visible: showSignatureLine
419 anchors {
420 top: headerContainer.bottom
421 left: parent.left
422 right: parent.right
423 bottom: parent.bottom
424 }
425
426 color: root.scopeStyle ? root.scopeStyle.headerDividerColor : "#e0e0e0"
427
428 Rectangle {
429 anchors {
430 top: parent.top
431 left: parent.left
432 right: parent.right
433 }
434 height: units.dp(1)
435 color: Qt.darker(parent.color, 1.1)
436 }
437 }
438
439 Row {
440 visible: bottomBorder.visible
441 spacing: units.gu(.5)434 spacing: units.gu(.5)
442 Repeater {435 Repeater {
443 objectName: "paginationRepeater"436 objectName: "paginationRepeater"
444 model: root.paginationCount437 model: root.paginationCount
445 Image {438 Rectangle {
446 objectName: "paginationDots_" + index439 objectName: "paginationDots_" + index
447 height: units.gu(1)440 height: units.gu(1)
448 width: height441 width: height
449 source: (index == root.paginationIndex) ? "graphics/pagination_dot_on.png" : "graphics/pagination_dot_off.png"442 radius: height / 2
443 color: index == root.paginationIndex ? UbuntuColors.blue : "transparent"
444 border.width: index == root.paginationIndex ? 0 : 1 // yes, one pixel and not 1dp
445 border.color: theme.palette.normal.baseText
450 }446 }
451 }447 }
452 anchors {
453 top: headerContainer.bottom
454 horizontalCenter: headerContainer.horizontalCenter
455 topMargin: units.gu(.5)
456 }
457 }
458
459 // FIXME this doesn't work with solid scope backgrounds due to z-ordering
460 Item {
461 id: bottomHighlight
462 visible: bottomBorder.visible
463 anchors {
464 top: parent.bottom
465 left: parent.left
466 right: parent.right
467 }
468 z: 1
469 height: units.dp(1)
470 opacity: 0.6
471
472 Rectangle {
473 anchors.fill: parent
474 color: if (root.scopeStyle) {
475 Qt.lighter(Qt.rgba(root.scopeStyle.background.r,
476 root.scopeStyle.background.g,
477 root.scopeStyle.background.b, 1.0), 1.2);
478 } else "#CCFFFFFF"
479 }
480 }448 }
481}449}
482450
=== modified file 'qml/Dash/GenericScopeView.qml'
--- qml/Dash/GenericScopeView.qml 2016-09-22 07:37:21 +0000
+++ qml/Dash/GenericScopeView.qml 2016-10-14 09:08:35 +0000
@@ -644,6 +644,7 @@
644 scopeHasFilters: scopeView.scope.filters != null644 scopeHasFilters: scopeView.scope.filters != null
645 activeFiltersCount: scopeView.scope.activeFiltersCount645 activeFiltersCount: scopeView.scope.activeFiltersCount
646 showBackButton: scopeView.hasBackAction646 showBackButton: scopeView.hasBackAction
647 showSignatureLine: !showBackButton
647 searchEntryEnabled: true648 searchEntryEnabled: true
648 settingsEnabled: scopeView.scope && scopeView.scope.settings && scopeView.scope.settings.count > 0 || false649 settingsEnabled: scopeView.scope && scopeView.scope.settings && scopeView.scope.settings.count > 0 || false
649 favoriteEnabled: scopeView.scope && scopeView.scope.id !== "clickscope"650 favoriteEnabled: scopeView.scope && scopeView.scope.id !== "clickscope"
650651
=== removed file 'qml/Dash/graphics/overview_hint@27.png'
651Binary files qml/Dash/graphics/overview_hint@27.png 2016-06-03 23:00:28 +0000 and qml/Dash/graphics/overview_hint@27.png 1970-01-01 00:00:00 +0000 differ652Binary files qml/Dash/graphics/overview_hint@27.png 2016-06-03 23:00:28 +0000 and qml/Dash/graphics/overview_hint@27.png 1970-01-01 00:00:00 +0000 differ
=== added file 'qml/Dash/graphics/overview_hint@30.png'
652Binary files qml/Dash/graphics/overview_hint@30.png 1970-01-01 00:00:00 +0000 and qml/Dash/graphics/overview_hint@30.png 2016-10-14 09:08:35 +0000 differ653Binary files qml/Dash/graphics/overview_hint@30.png 1970-01-01 00:00:00 +0000 and qml/Dash/graphics/overview_hint@30.png 2016-10-14 09:08:35 +0000 differ
=== removed file 'qml/Dash/graphics/pagination_dot_off@30.png'
653Binary files qml/Dash/graphics/pagination_dot_off@30.png 2016-06-03 23:00:28 +0000 and qml/Dash/graphics/pagination_dot_off@30.png 1970-01-01 00:00:00 +0000 differ654Binary files qml/Dash/graphics/pagination_dot_off@30.png 2016-06-03 23:00:28 +0000 and qml/Dash/graphics/pagination_dot_off@30.png 1970-01-01 00:00:00 +0000 differ
=== removed file 'qml/Dash/graphics/pagination_dot_on@30.png'
654Binary files qml/Dash/graphics/pagination_dot_on@30.png 2016-06-03 23:00:28 +0000 and qml/Dash/graphics/pagination_dot_on@30.png 1970-01-01 00:00:00 +0000 differ655Binary files qml/Dash/graphics/pagination_dot_on@30.png 2016-06-03 23:00:28 +0000 and qml/Dash/graphics/pagination_dot_on@30.png 1970-01-01 00:00:00 +0000 differ
=== removed file 'qml/Dash/graphics/paper.png'
655Binary files qml/Dash/graphics/paper.png 2016-06-03 23:00:28 +0000 and qml/Dash/graphics/paper.png 1970-01-01 00:00:00 +0000 differ656Binary files qml/Dash/graphics/paper.png 2016-06-03 23:00:28 +0000 and qml/Dash/graphics/paper.png 1970-01-01 00:00:00 +0000 differ
=== modified file 'tests/qmltests/Dash/tst_DashPageHeader.qml'
--- tests/qmltests/Dash/tst_DashPageHeader.qml 2016-08-03 15:51:32 +0000
+++ tests/qmltests/Dash/tst_DashPageHeader.qml 2016-10-14 09:08:35 +0000
@@ -173,9 +173,9 @@
173 for (var j=0; j<paginationRepeater.count; j++) {173 for (var j=0; j<paginationRepeater.count; j++) {
174 var paginationDot = findChild(pageHeader, "paginationDots_"+j);174 var paginationDot = findChild(pageHeader, "paginationDots_"+j);
175 if (i==j) {175 if (i==j) {
176 compare(paginationDot.source.toString().indexOf("pagination_dot_on") > -1, true);176 compare(paginationDot.color, UbuntuColors.blue);
177 } else {177 } else {
178 compare(paginationDot.source.toString().indexOf("pagination_dot_off") > -1, true);178 compare(paginationDot.color.a, 0);
179 }179 }
180 }180 }
181 }181 }
182182
=== modified file 'tests/qmltests/Dash/tst_GenericScopeView.qml'
--- tests/qmltests/Dash/tst_GenericScopeView.qml 2016-09-22 07:41:49 +0000
+++ tests/qmltests/Dash/tst_GenericScopeView.qml 2016-10-14 09:08:35 +0000
@@ -485,7 +485,7 @@
485 mouseClick(seeAll0);485 mouseClick(seeAll0);
486 verify(category0.expanded);486 verify(category0.expanded);
487 tryCompare(category0, "height", category0.item.expandedHeight + seeAll0.height);487 tryCompare(category0, "height", category0.item.expandedHeight + seeAll0.height);
488 tryCompare(genericScopeView.categoryView, "contentY", units.gu(8));488 tryCompare(genericScopeView.categoryView, "contentY", units.gu(8.5));
489489
490 scrollToEnd();490 scrollToEnd();
491491

Subscribers

People subscribed via source and target branches