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

Proposed by Albert Astals Cid
Status: Merged
Approved by: Lukáš Tinkl
Approved revision: 2597
Merged at revision: 2628
Proposed branch: lp:~aacid/unity8/splash_spinner_vertical_tweak
Merge into: lp:unity8
Diff against target: 72 lines (+19/-8)
3 files modified
qml/Stages/Splash.qml (+3/-6)
tests/mocks/Unity/Application/ApplicationInfo.cpp (+1/-1)
tests/qmltests/Stages/tst_Splash.qml (+15/-1)
To merge this branch: bzr merge lp:~aacid/unity8/splash_spinner_vertical_tweak
Reviewer Review Type Date Requested Status
Unity8 CI Bot continuous-integration Needs Fixing
Lukáš Tinkl (community) Approve
Review via email: mp+303021@code.launchpad.net

Commit message

Make sure the spinner does not get too close to the title text

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?
No, but should be fine as it makes the spinner position a bit less hardcoded.

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

FAILED: Continuous integration, rev:2595
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1964/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/2563
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1387
    FAILURE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1387/console
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1387
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2591
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2471
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2471
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2471
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2465
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2465/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2465
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2465/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2465
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2465/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2465
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2465/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2465
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2465/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2465
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2465/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2465
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2465/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2465
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2465/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2465
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2465/artifact/output/*zip*/output.zip

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

review: Needs Fixing (continuous-integration)
2596. By Albert Astals Cid

Fix path

2597. By Albert Astals Cid

Add landscape checkbox

Revision history for this message
Lukáš Tinkl (lukas-kde) wrote :

Yup, works fine for me

* 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.

No, unrelated failures

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

FAILED: Continuous integration, rev:2597
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/2011/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/2636
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/1441
    UNSTABLE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial+overlay,testname=qmluitests.sh/1441
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=yakkety,testname=qmluitests.sh/1441
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2664
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2538
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2538
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2538
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2532
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2532/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2532
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2532/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2532
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2532/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2532
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2532/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2532
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2532/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2532
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2532/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2532
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2532/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2532
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2532/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2532
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2532/artifact/output/*zip*/output.zip

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

review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'qml/Stages/Splash.qml'
--- qml/Stages/Splash.qml 2016-05-19 15:22:34 +0000
+++ qml/Stages/Splash.qml 2016-08-23 09:06:36 +0000
@@ -119,6 +119,7 @@
119 }119 }
120120
121 Label {121 Label {
122 id: titleLabel
122 text: root.title123 text: root.title
123 anchors.horizontalCenter: parent.horizontalCenter124 anchors.horizontalCenter: parent.horizontalCenter
124 anchors.top: iconShape.bottom125 anchors.top: iconShape.bottom
@@ -137,12 +138,8 @@
137138
138 ActivityIndicator {139 ActivityIndicator {
139 id: spinner140 id: spinner
140 anchors.centerIn: header.visible ? parent : undefined141 anchors.centerIn: parent
141 anchors.verticalCenterOffset: header.visible ? header.height / 2 : 0142 anchors.verticalCenterOffset: header.visible ? header.height / 2 : Math.max(titleLabel.y / 2, units.gu(8))
142
143 anchors.horizontalCenter: header.visible ? undefined : parent.horizontalCenter
144 anchors.bottom: header.visible ? undefined : parent.bottom
145 anchors.bottomMargin: header.visible ? 0 : units.gu(12)
146 }143 }
147144
148 MouseArea {145 MouseArea {
149146
=== modified file 'tests/mocks/Unity/Application/ApplicationInfo.cpp'
--- tests/mocks/Unity/Application/ApplicationInfo.cpp 2016-07-25 14:57:11 +0000
+++ tests/mocks/Unity/Application/ApplicationInfo.cpp 2016-08-23 09:06:36 +0000
@@ -138,7 +138,7 @@
138138
139void ApplicationInfo::setIconId(const QString &iconId)139void ApplicationInfo::setIconId(const QString &iconId)
140{140{
141 setIcon(QString("../../tests/graphics/applicationIcons/%2@18.png")141 setIcon(QString("../../graphics/applicationIcons/%2@18.png")
142 .arg(iconId));142 .arg(iconId));
143}143}
144144
145145
=== modified file 'tests/qmltests/Stages/tst_Splash.qml'
--- tests/qmltests/Stages/tst_Splash.qml 2015-07-15 15:07:19 +0000
+++ tests/qmltests/Stages/tst_Splash.qml 2016-08-23 09:06:36 +0000
@@ -55,7 +55,7 @@
55 bottom: parent.bottom55 bottom: parent.bottom
56 left: parent.left56 left: parent.left
57 }57 }
58 width: units.gu(40)58 width: landscapeCheckbox.checked ? units.gu(70) : units.gu(40)
59 sourceComponent: splashComponent59 sourceComponent: splashComponent
60 }60 }
6161
@@ -97,6 +97,20 @@
97 CheckBox {id: footerColorCheckbox; checked: false; }97 CheckBox {id: footerColorCheckbox; checked: false; }
98 Label { text: "custom footerColor"; anchors.verticalCenter: parent.verticalCenter }98 Label { text: "custom footerColor"; anchors.verticalCenter: parent.verticalCenter }
99 }99 }
100 Row {
101 anchors { left: parent.left; right: parent.right }
102 CheckBox {
103 id: landscapeCheckbox
104 checked: false
105 onCheckedChanged: {
106 if (checked) {
107 window.width = Math.max(window.width, units.gu(110));
108 }
109 window.height = checked ? units.gu(40) : units.gu(70);
110 }
111 }
112 Label { text: "Landscape"; anchors.verticalCenter: parent.verticalCenter }
113 }
100 }114 }
101 }115 }
102116

Subscribers

People subscribed via source and target branches