Merge lp:~robertcarr/unity/homelauncher-dnd into lp:unity

Proposed by Robert Carr
Status: Merged
Merged at revision: 1534
Proposed branch: lp:~robertcarr/unity/homelauncher-dnd
Merge into: lp:unity
Diff against target: 18 lines (+4/-2)
1 file modified
plugins/unityshell/src/BamfLauncherIcon.cpp (+4/-2)
To merge this branch: bzr merge lp:~robertcarr/unity/homelauncher-dnd
Reviewer Review Type Date Requested Status
Unity Team Pending
Review via email: mp+75012@code.launchpad.net

Description of the change

Let the home launcher accept all drag types.

To post a comment you must log in.
Revision history for this message
Andrea Azzarone (azzar1) wrote :

Please use is_home_launcher instead of IsHomeLauncher.

Revision history for this message
Robert Carr (robertcarr) wrote :

Whooooops :) fixed

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plugins/unityshell/src/BamfLauncherIcon.cpp'
--- plugins/unityshell/src/BamfLauncherIcon.cpp 2011-09-08 11:04:17 +0000
+++ plugins/unityshell/src/BamfLauncherIcon.cpp 2011-09-12 15:01:26 +0000
@@ -1064,10 +1064,12 @@
1064std::set<std::string> BamfLauncherIcon::ValidateUrisForLaunch(unity::DndData& uris)1064std::set<std::string> BamfLauncherIcon::ValidateUrisForLaunch(unity::DndData& uris)
1065{1065{
1066 std::set<std::string> result;1066 std::set<std::string> result;
1067 1067 gboolean is_home_launcher = g_str_has_suffix(DesktopFile(), "nautilus-home.desktop");
1068
1068 for (auto i : uris.Types())1069 for (auto i : uris.Types())
1069 for (auto j : GetSupportedTypes())1070 for (auto j : GetSupportedTypes())
1070 if (g_content_type_is_a(i.c_str(), j.c_str()))1071 if (g_content_type_is_a(i.c_str(), j.c_str()) ||
1072 is_home_launcher)
1071 {1073 {
1072 for (auto k : uris.UrisByType(i))1074 for (auto k : uris.UrisByType(i))
1073 result.insert(k);1075 result.insert(k);