Merge lp:~mzanetti/unity8/launcher-dont-hide-on-focus into lp:unity8

Proposed by Michael Zanetti
Status: Merged
Approved by: Michael Zanetti
Approved revision: 306
Merged at revision: 319
Proposed branch: lp:~mzanetti/unity8/launcher-dont-hide-on-focus
Merge into: lp:unity8
Diff against target: 12 lines (+1/-1)
1 file modified
Shell.qml (+1/-1)
To merge this branch: bzr merge lp:~mzanetti/unity8/launcher-dont-hide-on-focus
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Nicolas d'Offay (community) Approve
Review via email: mp+184741@code.launchpad.net

Commit message

Don't hide the launcher on changes in the stages.

Description of the change

Right now, when you launch an app and then quickly reveal the launcher while the app is still loading, the launcher will be hidden. This breaks usability and is not really needed.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Nicolas d'Offay (nicolas-doffay) wrote :

LGTM

review: Approve
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/433/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/generic-cleanup-mbs/2195
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-saucy/3356
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-touch/893
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-saucy/1686
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-saucy-amd64-autolanding/14
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-saucy-armhf-autolanding/432
        deb: http://jenkins.qa.ubuntu.com/job/unity8-saucy-armhf-autolanding/432/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-saucy-i386-autolanding/433
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-i386/3364
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-i386/3364/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-saucy/2834
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/895
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/895/artifact/work/output/*zip*/output.zip
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/743
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/753

review: Needs Fixing (continuous-integration)
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 'Shell.qml'
2--- Shell.qml 2013-09-10 08:10:10 +0000
3+++ Shell.qml 2013-09-10 09:26:22 +0000
4@@ -255,7 +255,7 @@
5 property bool fullyShown: shown && x == 0 && parent.x == 0
6 property bool fullyHidden: !shown && x == width
7 available: !greeter.shown
8- hides: [launcher, panel.indicators]
9+ hides: [panel.indicators]
10 shown: false
11 opacity: 1.0
12 showAnimation: StandardAnimation { property: "x"; duration: 350; to: 0; easing.type: Easing.OutCubic }

Subscribers

People subscribed via source and target branches