Merge lp:~mzanetti/unity8/rotate-dash into lp:unity8

Proposed by Michael Zanetti
Status: Merged
Approved by: Michael Zanetti
Approved revision: 2334
Merged at revision: 2365
Proposed branch: lp:~mzanetti/unity8/rotate-dash
Merge into: lp:unity8
Diff against target: 8 lines (+0/-1)
1 file modified
data/unity8-dash.desktop.in (+0/-1)
To merge this branch: bzr merge lp:~mzanetti/unity8/rotate-dash
Reviewer Review Type Date Requested Status
Daniel d'Andrada (community) Needs Fixing
Lukáš Tinkl (community) Approve
Unity8 CI Bot continuous-integration Needs Fixing
Review via email: mp+292377@code.launchpad.net

Commit message

Drop the rotation lock for the dash

Description of the change

Everyone thinks it's a bug and it really should rotate. Design agrees but there is no point in waiting for the next dash rework spec to be finalized just to enable this. I've had it enabled for a while on my device and it seems to behave well.

 * Are there any related MPs required for this MP to build/function as expected? Please list.
no
 * Did you perform an exploratory manual test run of your code change and any related functionality?
yes
 * Did you make sure that your branch does not contain spurious tags?
yes
 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
no
 * If you changed the UI, has there been a design review?
Yes, Paty acked it.

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

FAILED: Continuous integration, rev:2334
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1027/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=vivid+overlay,testname=qmluitests.sh/584
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=amd64,release=xenial,testname=qmluitests.sh/584
    FAILURE: https://unity8-jenkins.ubuntu.com/job/test-0-autopkgtest/label=phone-armhf,release=vivid+overlay,testname=autopilot.sh/584/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/1375
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/1345
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial/1345
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/1345
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1345
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1345/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial/1345
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial/1345/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1345
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1345/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1345
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1345/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/1345
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial/1345/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1345
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1345/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/1345
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/1345/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/1345
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial/1345/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/1345
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/1345/artifact/output/*zip*/output.zip

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

review: Needs Fixing (continuous-integration)
Revision history for this message
Lukáš Tinkl (lukas-kde) wrote :

Approving, based on feedback we got from design

review: Approve
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

We should update the mock dash accordingly (make it rotate as well). Might break some tests relying on a non-rotating dash, so have to run them to make sure.

review: Needs Fixing
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

> We should update the mock dash accordingly (make it rotate as well). Might
> break some tests relying on a non-rotating dash, so have to run them to make
> sure.

And update such tests as well, if needed.

Revision history for this message
Michael Zanetti (mzanetti) wrote :

This branch is in a silo that's already about to be handed off to QA. I'll update the mock in another branch. We really want to get this to the users in OTA-11

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/unity8-dash.desktop.in'
2--- data/unity8-dash.desktop.in 2015-10-30 16:17:46 +0000
3+++ data/unity8-dash.desktop.in 2016-04-20 13:33:00 +0000
4@@ -7,4 +7,3 @@
5 Icon=@CMAKE_INSTALL_PREFIX@/@SHELL_APP_DIR@/Dash/unity8-dash.png
6 NoDisplay=true
7 X-Ubuntu-Touch=true
8-X-Ubuntu-Supported-Orientations=primary

Subscribers

People subscribed via source and target branches