Merge lp:~mterry/unity8/infographic-white into lp:unity8

Proposed by Michael Terry
Status: Merged
Approved by: Albert Astals Cid
Approved revision: 2571
Merged at revision: 2646
Proposed branch: lp:~mterry/unity8/infographic-white
Merge into: lp:unity8
Prerequisite: lp:~mterry/unity8/infographic-refresh
Diff against target: 69 lines (+2/-14)
3 files modified
qml/Greeter/CoverPage.qml (+0/-2)
qml/Greeter/Infographics.qml (+2/-10)
qml/Greeter/WideView.qml (+0/-2)
To merge this branch: bzr merge lp:~mterry/unity8/infographic-white
Reviewer Review Type Date Requested Status
Albert Astals Cid (community) Approve
Unity8 CI Bot continuous-integration Needs Fixing
Review via email: mp+305128@code.launchpad.net

Commit message

Make infographic bubbles white even on the default wallpaper.

Description of the change

Make infographic bubbles white even on the default wallpaper.

This was a request from Alex M in Design. He liked the white infographics uses on custom wallpapers and felt it was better than our colored bubbles even for the default wallpaper.

This means we no longer pay attention to the color set in libusermetrics.

 * Are there any related MPs required for this MP to build/function as expected? Please list.
 No, just pre-reqs

 * 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?
 NA

 * If you changed the UI, has there been a design review?
 Not really, but it is a direct request and he knows what it will look like already.

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

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

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

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

 * 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.
Unrelated launcher failures

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'qml/Greeter/CoverPage.qml'
2--- qml/Greeter/CoverPage.qml 2016-09-07 17:19:09 +0000
3+++ qml/Greeter/CoverPage.qml 2016-09-07 17:19:10 +0000
4@@ -109,8 +109,6 @@
5 model: root.infographicModel
6 clip: true // clip large data bubbles
7
8- useColor: !root.hasCustomBackground
9-
10 anchors {
11 verticalCenter: parent.verticalCenter
12 left: parent.left
13
14=== modified file 'qml/Greeter/Infographics.qml'
15--- qml/Greeter/Infographics.qml 2016-09-07 17:19:09 +0000
16+++ qml/Greeter/Infographics.qml 2016-09-07 17:19:10 +0000
17@@ -24,7 +24,6 @@
18 property var model
19
20 property int animDuration: 10
21- property bool useColor: true
22
23 QtObject {
24 id: d
25@@ -36,7 +35,6 @@
26 || circleChangeAnimTimer.running
27 }
28
29- // Define a replacement white-based color theme when useColor is false
30 QtObject {
31 id: whiteTheme
32 property color main: "white"
33@@ -177,10 +175,7 @@
34 ColorAnimation {
35 target: pastCircle
36 property: "color"
37- to: Gradient.threeColorByIndex(index, count,
38- useColor ?
39- infographic.model.secondColor :
40- whiteTheme)
41+ to: Gradient.threeColorByIndex(index, count, whiteTheme)
42 easing.type: Easing.OutCurve
43 duration: circleChangeAnimTimer.interval * d.circleModifier
44 }
45@@ -242,10 +237,7 @@
46 ColorAnimation {
47 target: presentCircle
48 property: "color"
49- to: Gradient.threeColorByIndex(index, infographic.model.currentDay,
50- useColor ?
51- infographic.model.firstColor :
52- whiteTheme)
53+ to: Gradient.threeColorByIndex(index, infographic.model.currentDay, whiteTheme)
54 easing.type: Easing.OutCurve
55 duration: circleChangeAnimTimer.interval * d.circleModifier
56 }
57
58=== modified file 'qml/Greeter/WideView.qml'
59--- qml/Greeter/WideView.qml 2016-09-07 17:19:09 +0000
60+++ qml/Greeter/WideView.qml 2016-09-07 17:19:10 +0000
61@@ -111,8 +111,6 @@
62 infographics {
63 height: 0.75 * parent.height
64 anchors.leftMargin: loginList.x + loginList.width
65-
66- useColor: !root.hasCustomBackground
67 }
68
69 onTease: root.tease()

Subscribers

People subscribed via source and target branches