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

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 :
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
=== modified file 'launcher/ApplicationLauncherIcon.cpp'
--- launcher/ApplicationLauncherIcon.cpp 2013-09-13 06:31:30 +0000
+++ launcher/ApplicationLauncherIcon.cpp 2013-10-02 13:48:30 +0000
@@ -198,6 +198,7 @@
198 if (!IsSticky())198 if (!IsSticky())
199 {199 {
200 SetQuirk(Quirk::VISIBLE, false);200 SetQuirk(Quirk::VISIBLE, false);
201 HideTooltip();
201202
202 /* Use a timeout to remove the icon, this avoids203 /* Use a timeout to remove the icon, this avoids
203 * that we remove an application that is going204 * that we remove an application that is going