Unity

Merge lp:~azzar1/unity/fix-1172769 into lp:unity

  1. fix-1172769
  2. Merge into trunk
Proposed by Andrea Azzarone
Status: Merged
Approved by: Brandon Schaefer
Approved revision: no longer in the source branch.
Merged at revision: 3549
Proposed branch: lp:~azzar1/unity/fix-1172769
Merge into: lp:unity
Diff against target: 11 lines (+1/-0)
1 file modified
launcher/ApplicationLauncherIcon.cpp (+1/-0)
To merge this branch: bzr merge lp:~azzar1/unity/fix-1172769
Reviewer Review Type Date Requested Status
Brandon Schaefer (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+188841@code.launchpad.net

Commit message

Hide the tooltip when an app is closed.

Description of the change

== Problem ==
Hide the tooltip when an app is closed.

== Fix ==
Hide the tooltip when an app is closed.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:3548
http://jenkins.qa.ubuntu.com/job/unity-ci/412/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-saucy-amd64-ci/301
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-saucy-armhf-ci/299
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-saucy-i386-ci/300

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-ci/412/rebuild

review: Approve (continuous-integration)
Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'launcher/ApplicationLauncherIcon.cpp'
2--- launcher/ApplicationLauncherIcon.cpp 2013-09-13 06:31:30 +0000
3+++ launcher/ApplicationLauncherIcon.cpp 2013-10-02 13:48:30 +0000
4@@ -198,6 +198,7 @@
5 if (!IsSticky())
6 {
7 SetQuirk(Quirk::VISIBLE, false);
8+ HideTooltip();
9
10 /* Use a timeout to remove the icon, this avoids
11 * that we remove an application that is going