Merge lp:~3v1n0/unity/dnd-icon-dash-hide into lp:unity

Proposed by Marco Trevisan (Treviño)
Status: Merged
Approved by: Andrea Azzarone
Approved revision: 3845
Merged at revision: 3855
Proposed branch: lp:~3v1n0/unity/dnd-icon-dash-hide
Merge into: lp:unity
Diff against target: 19 lines (+2/-0)
1 file modified
launcher/ApplicationLauncherIcon.cpp (+2/-0)
To merge this branch: bzr merge lp:~3v1n0/unity/dnd-icon-dash-hide
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Andrea Azzarone (community) Approve
Review via email: mp+228844@code.launchpad.net

Commit message

ApplicationLauncherIcon: make sure we close the dash if DnD is accepted

And we're about to focus the application window(s)

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
Andrea Azzarone (azzar1) wrote :

LGTM.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

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 2014-06-07 16:27:16 +0000
3+++ launcher/ApplicationLauncherIcon.cpp 2014-07-30 13:18:26 +0000
4@@ -33,6 +33,7 @@
5 #include "MultiMonitor.h"
6 #include "unity-shared/DesktopApplicationManager.h"
7 #include "unity-shared/GnomeFileManager.h"
8+#include "unity-shared/UBusWrapper.h"
9 #include "unity-shared/UBusMessages.h"
10 #include "unity-shared/UScreen.h"
11
12@@ -1168,6 +1169,7 @@
13 if (!IsRunning())
14 return false;
15
16+ UBusManager::SendMessage(UBUS_OVERLAY_CLOSE_REQUEST);
17 Focus(ActionArg(ActionArg::Source::LAUNCHER, 1, timestamp));
18
19 if (GetWindows(WindowFilter::ON_CURRENT_DESKTOP).size() > 1)