Merge lp:~tcfox54-gmail/ubuntu/precise/rhythmbox/add_quicklist into lp:ubuntu/precise/rhythmbox

[Trenton Fox]

Adds an Unity quicklist for play, pause, next, previous, and stop.
Fixed the quicklist to meet specifications.

[Michael Hall (mhall119)]

Thanks so much for your submission!

Recently a new FreeDesktop.org specification[1] has been approved for providing desktop-agnostic shortcuts. The Quicklist documentation has been updated to coincide with this new spec[2].

Since this is now an official standard, and will benefit environments other than Unity, we encourage you to change your code to match. This will require the following changes:
1) Change "X-Ayatana-Desktop-Shortcuts" to just "Actions"
2) Change each section from "[<action> Shortcut Group]" to "[Desktop Action <action>]"
3) Remove "TargetEnvironment=Unity"
4) Change your Name= entry to use Title Case[3] (capitalizing important words)

Please update this merge proposal with these changes so they can be included in the upcoming release of Ubuntu. Please also make sure you submit them to the upstream developers so they can be included there in future versions.

[1] http://standards.freedesktop.org/desktop-entry-spec/latest/ar01s08.html
[2] https://wiki.ubuntu.com/Unity/LauncherAPI#Quicklists
[3] http://en.wikipedia.org/wiki/Title_case#Headings_and_publication_titles

review: Needs Fixing

[Sebastien Bacher (seb128)]

Setting to "Work in progress" please set back to "Needs review" when you update so it shows again on the sponsoring list

[Trenton Fox (tcfox54-gmail)]

I have pulled the branch down but do not know how to update it to this merge request. Could either of you point me in the right direction? I would really love to get the quicklist into 12.04.

[Michael Hall (mhall119)]

If you just bzr push lp:~tcfox54-gmail/ubuntu/precise/rhythmbox/add_quicklist again, it will update this merge proposal with the latest revision.

[Trenton Fox (tcfox54-gmail)]

I initially tried this, but it came up as diverged. I went through the
instructions to fix that and tried to push it again. However, it said it
was still diverged. :/

On Sun, Mar 11, 2012 at 1:34 PM, Michael Hall <email address hidden> wrote:

> If you just bzr push
> lp:~tcfox54-gmail/ubuntu/precise/rhythmbox/add_quicklist again, it will
> update this merge proposal with the latest revision.
> --
>
> https://code.launchpad.net/~tcfox54-gmail/ubuntu/precise/rhythmbox/add_quicklist/+merge/94266<https://code.launchpad.net/%7Etcfox54-gmail/ubuntu/precise/rhythmbox/add_quicklist/+merge/94266>
> You are the owner of
> lp:~tcfox54-gmail/ubuntu/precise/rhythmbox/add_quicklist.
>

[Michael Hall (mhall119)]

Ah, right, because you did an uncommit and then another commit, they will have diverged. If you add --overwrite to your bzr push, it'll replace what's here with your new branch.

[Michael Terry (mterry)]

Thanks for the patch! I pushed it in and filed an upstream bug about it. https://bugzilla.gnome.org/show_bug.cgi?id=672180

=== added directory 'data/debian'
=== added directory 'data/debian/patches'
=== modified file 'data/rhythmbox.desktop.in.in'
--- data/rhythmbox.desktop.in.in	2010-08-24 17:19:03 +0000
+++ data/rhythmbox.desktop.in.in	2012-03-11 23:02:17 +0000
@@ -16,3 +16,25 @@
 X-GNOME-Bugzilla-Component=general
 X-GNOME-Bugzilla-OtherBinaries=rhythmbox-client;rhythmbox-metadata;
 X-GNOME-Bugzilla-Version=@VERSION@
+Actions=Play;Pause;Next;Previous;Stop;
+
+[Desktop Action Play]
+Name=Play
+Exec=rhythmbox-client --play
+
+[Desktop Action Pause]
+Name=Pause
+Exec=rhythmbox-client --pause
+
+[Desktop Action Next]
+Name=Next
+Exec=rhythmbox-client --next
+
+[Desktop Action Previous]
+Name=Previous
+Exec=rhythmbox-client --previous
+
+[Desktop Action Stop]
+Name=Stop
+Exec=rhythmbox-client --stop
+