Merge lp:~jeremywootten/slingshot/fix-1565134-cannot-drag-from-search into lp:~elementary-pantheon/slingshot/trunk

[Jeremy Wootten]

This branch implements drag handling by the search view.

Dragging between Slingshot search view and Plank functions. Slingshot closes on drag end.

[lp:~jeremywootten/slingshot/fix-1565134-cannot-drag-from-search]

660. By Jeremy Wootten on 2016-06-15

Enable drag from search view

[Jeremy Wootten (jeremywootten)]

r660 overwritten with amendments to deal with rows without an app uri.

=== modified file 'src/Widgets/SearchItem.vala'
--- src/Widgets/SearchItem.vala	2016-04-21 18:23:20 +0000
+++ src/Widgets/SearchItem.vala	2016-06-15 13:30:05 +0000
@@ -39,10 +39,9 @@
         public Backend.App app { get; construct; }
         public ResultType result_type { public get; construct; }
 
-        public bool dragging = false; //prevent launching
-
         private Gtk.Label name_label;
-        private Gtk.Image icon;
+        public Gtk.Image icon { public get; private set; }
+        public string? app_uri { get; private set; }
         private Cancellable? cancellable = null;
 
         public SearchItem (Backend.App app, string search_term = "", ResultType result_type = ResultType.UNKNOWN) {
@@ -89,24 +88,14 @@
 
             add (grid);
 
-            if (result_type != SearchItem.ResultType.APP_ACTIONS)
+            if (result_type != SearchItem.ResultType.APP_ACTIONS) {
                 launch_app.connect (app.launch);
+            }
 
+            app_uri = null;
             var app_match = app.match as Synapse.ApplicationMatch;
             if (app_match != null) {
-                Gtk.TargetEntry dnd = {"text/uri-list", 0, 0};
-                Gtk.drag_source_set (this, Gdk.ModifierType.BUTTON1_MASK, {dnd},
-                Gdk.DragAction.COPY);
-                this.drag_begin.connect ( (ctx) => {
-                    this.dragging = true;
-                    Gtk.drag_set_icon_gicon (ctx, app.icon, 0, 0);
-                });
-                this.drag_end.connect ( () => {
-                    this.dragging = false;
-                });
-                this.drag_data_get.connect ( (ctx, sel, info, time) => {
-                    sel.set_uris ({File.new_for_path (app_match.filename).get_uri ()});
-                });
+                app_uri = File.new_for_path (app_match.filename).get_uri ();
             }
         }
 

=== modified file 'src/Widgets/SearchView.vala'
--- src/Widgets/SearchView.vala	2016-05-04 02:46:59 +0000
+++ src/Widgets/SearchView.vala	2016-06-15 13:30:05 +0000
@@ -33,6 +33,9 @@
         private Gtk.ListBox list_box;
         Gee.HashMap<SearchItem.ResultType, uint> limitator;
 
+        private bool dragging = false;
+        private string? drag_uri = null;
+
         public SearchView () {
 
         }
@@ -46,15 +49,55 @@
             list_box.activate_on_single_click = true;
             list_box.set_sort_func ((row1, row2) => update_sort (row1, row2));
             list_box.set_header_func ((Gtk.ListBoxUpdateHeaderFunc) update_header);
+            list_box.set_selection_mode (Gtk.SelectionMode.BROWSE);
             list_box.row_activated.connect ((row) => {
                 var search_item = row as SearchItem;
-                if (search_item.result_type == SearchItem.ResultType.APP_ACTIONS || search_item.result_type == SearchItem.ResultType.LINK) {
-                    search_item.app.match.execute (null);
-                } else {
-                    search_item.app.launch ();
-                }
-
-                app_launched ();
+                if (!dragging) {
+                    if (search_item.result_type == SearchItem.ResultType.APP_ACTIONS || search_item.result_type == SearchItem.ResultType.LINK) {
+                        search_item.app.match.execute (null);
+                    } else {
+                        search_item.app.launch ();
+                    }
+
+                    app_launched ();
+                }
+            });
+
+            // Drag support
+            Gtk.TargetEntry dnd = {"text/uri-list", 0, 0};
+            Gtk.drag_source_set (list_box, Gdk.ModifierType.BUTTON1_MASK, {dnd}, Gdk.DragAction.COPY);
+
+            list_box.motion_notify_event.connect ((event) => {
+                if (!dragging) {
+                    list_box.select_row (list_box.get_row_at_y ((int)event.y));
+                }
+                return false;
+            });
+
+            list_box.drag_begin.connect ( (ctx) => {
+                var sr = list_box.get_selected_rows ();
+                if (sr.length () > 0) {
+                    var di = (SearchItem)(sr.first ().data);
+                    drag_uri = di.app_uri;
+                    if (drag_uri != null);
+                        dragging = true;
+                        Gtk.drag_set_icon_gicon (ctx, di.icon.gicon, 0, 0);
+                    }
+                }
+            });
+
+            list_box.drag_end.connect ( () => {
+                if (drag_uri != null) {
+                    app_launched (); /* This causes indicator to close */
+                }
+                dragging = false;
+                drag_uri = null;
+            });
+
+            list_box.drag_data_get.connect ( (ctx, sel, info, time) => {
+                if (drag_uri != null) {
+                    sel.set_uris ({drag_uri});
+                }
             });
 
             // alert view