GNOME Bugzilla – Bug 80736
windows aren't raised when things are dragged over their buttons
Last modified: 2005-01-10 17:36:14 UTC
In GNOME 1, when something was dragged over a window list button, the corresponding entry was raised after about a second. The GNOME 2 window list, as of garnome 0.9.5, doesn't do this. I consider this a bug.
I think this feature(raising windows while hovering with a draged-object over their window list button) is really importent as otherwise drag and drop is made a lot more difficult. (MS had this starting from '95) Can someone add the keywords: 1.x-parity Usability To this bug?
Marking high due to the feature lossage and adding the requested keywords. Thanks, Martin.
Would need to implemented in libwnck
I marked this down because while it is a regression, it really isn't a particularly huge feature that we're missing. I'm of course open to differences of opinion on this but I don't see this as worthy of being 'high' unless someone patches it.
Created attachment 9223 [details] [review] Patch for activating windows when drag item is focused on the tasklist button.
hp: done only for ungrouped windows atm. Should it be done for grouped ones too, not sure exactly how to proceed on that though.
hp->poke() :)
It would be good to have drag over for the pager so you could drag between desktops. Should this be opened as a seperate bug or is it sufficiently related?
I'd also like to have the option to switch windows with A-Tab while dragging objects around...
havoc: would like to know if something is wrong in my patch. Please let me know, I will re-work on it.Thanks.
In the timeout function, you don't need gtk_timeout_remove, as returning FALSE will remove it. When the task list is destroyed you need to remove the timeout to avoid a crash. Should use GLib functions for the timeout instead of deprecated gtk_timeout_ functions. Looks about right though. Setting priority high.
Created attachment 10113 [details] [review] Patch with the changes Havoc suggested.
Patch looks fine to commit (add a space before paren in "if(task->button_activate" is all I see)
Havoc:Committed the patch in HEAD with the changes suggested. Thanks :)
See bug 91406 about handling dragging to a grouped button in the tasklist.
*** Bug 128664 has been marked as a duplicate of this bug. ***