GNOME Bugzilla – Bug 642567
Add Unity quicklist support
Last modified: 2011-09-26 21:08:19 UTC
Created attachment 181123 [details] [review] Add unity quicklist support In Unity, we have a feature that enables static Quicklist entries. Those entries are always displayed on right click once the application is added to the launcher. This patch adds two entries to gnome-screenshot quicklist in Unity: - take the whole screen - take the current window If you need any other information, do not hesitate.
Review of attachment 181123 [details] [review]: generally, I don't see anything bad about this. ::: gnome-utils-2.32.0.orig/gnome-screenshot/gnome-screenshot.desktop.in @@ +18,3 @@ +_Name=Take a screenshot of the whole screen +Exec=gnome-screenshot +OnlyShowIn=Unity ugh. this should be X-Unity, unless you amended the spec to add Unity to the list of platforms.
Sorry, I didn't realize there was a list of OnlyShowIn keys. I've submitted a patch to add Unity to that list. http://lists.freedesktop.org/archives/xdg/2011-February/011812.html Hopefully this can get resolved quickly. Thanks for looking at the patch!
Created attachment 184084 [details] [review] Add unity quicklist support From the discussion, a X- decision has been target as OnlyShowIn not suit Unity in this case. Updated the patch
Review of attachment 184084 [details] [review]: looks okay to me; if you want to land this before code freeze you'll have to do it now :-)
excellent, thanks a lot (I don't have gnome commit rights on GNOME git, so if you can land it :))
Comment on attachment 184084 [details] [review] Add unity quicklist support This will have to wait after we branch now; as it adds new strings.
Pushed now to master, as gnome-utils branched.