GNOME Bugzilla – Bug 609913
[Overview] Make indicators drop targets
Last modified: 2010-02-15 16:25:49 UTC
Opening apps on specific workspaces is easy in grid view, but not in linear view. An easy solution is to allow dropping on the workspace indicators. They might be small but from quick testing it seems to work well for me (well in fact they are bigger than the [+] button).
Created attachment 153760 [details] [review] [Overview] Make indicators drop targets
Comment on attachment 153760 [details] [review] [Overview] Make indicators drop targets >Subject: [PATCH] [Overview] Make indicators drop targets > >Allow dropping on the indicators to open apps on specific workspaces. say "workspace indicators", not just "indicators" please (in both places) >+ actor._delegate = actor; >+ actor._delegate.acceptDrop = Lang.bind(this, function(source, actor, x, y, time) { huh... not totally sure I like that. Although if there isn't already a javascript object per-indicator I guess it would be pointless to make a whole new class just for that. You should be able to do actor._delegate = {}; instead (and keep the second line as-is), which I think is probably nicer in terms of not having properties/functionality/whatever accidentally leak from 'actor' into its delegate. >+ this._workspaces[i]._metaWorkspace.activate(global.get_current_time()); >+ return this._workspaces[i].acceptDrop(source, actor, x, y, time) swap that; you should only activate the workspace if its acceptDrop method returns true.
Created attachment 153828 [details] [review] [Overview] Make workspace indicators drop targets Fixed up patch.
Comment on attachment 153828 [details] [review] [Overview] Make workspace indicators drop targets Pushed as a6df234.