Merge lp:~tintou/slingshot/dnd-grab into lp:~elementary-pantheon/slingshot/trunk

Proposed by Corentin Noël
Status: Rejected
Rejected by: Cody Garver
Proposed branch: lp:~tintou/slingshot/dnd-grab
Merge into: lp:~elementary-pantheon/slingshot/trunk
Diff against target: 25 lines (+6/-4)
1 file modified
src/Widgets/AppEntry.vala (+6/-4)
To merge this branch: bzr merge lp:~tintou/slingshot/dnd-grab
Reviewer Review Type Date Requested Status
Cody Garver (community) Needs Fixing
Review via email: mp+248209@code.launchpad.net

Description of the change

Resubmit

To post a comment you must log in.
Revision history for this message
Cody Garver (codygarver) wrote :

When I drag an icon to the dock, the Slingshot window refuses to close when I click outside of it. Only leaves when I click Applications on panel.

This does not happen with r501

review: Needs Fixing

Unmerged revisions

502. By Corentin Noël

Re-establish the device grab with the right device.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/Widgets/AppEntry.vala'
--- src/Widgets/AppEntry.vala 2015-02-01 22:32:07 +0000
+++ src/Widgets/AppEntry.vala 2015-02-02 00:14:10 +0000
@@ -92,15 +92,17 @@
92 return false;92 return false;
93 });93 });
9494
95 this.drag_begin.connect ( (ctx) => {95 this.drag_begin.connect ((ctx) => {
96 this.dragging = true;96 this.dragging = true;
97 Gtk.drag_set_icon_pixbuf (ctx, icon, 0, 0);97 Gtk.drag_set_icon_pixbuf (ctx, icon, 0, 0);
98 });98 });
9999
100 this.drag_end.connect ( () => {100 this.drag_end.connect ((ctx) => {
101 this.dragging = false;101 this.dragging = false;
102 var slingshot_app = (Gtk.Application) GLib.Application.get_default ();102 ctx.get_device ().grab (get_window (), Gdk.GrabOwnership.NONE, true,
103 ((SlingshotView)slingshot_app.active_window).grab_device ();103 Gdk.EventMask.SMOOTH_SCROLL_MASK | Gdk.EventMask.BUTTON_PRESS_MASK |
104 Gdk.EventMask.BUTTON_RELEASE_MASK | Gdk.EventMask.POINTER_MOTION_MASK,
105 null, Gdk.CURRENT_TIME);
104 });106 });
105107
106 this.drag_data_get.connect ( (ctx, sel, info, time) => {108 this.drag_data_get.connect ( (ctx, sel, info, time) => {

Subscribers

People subscribed via source and target branches