Merge lp:~nik90/ubuntu/precise/totem/add_quicklist into lp:ubuntu/precise/totem

[Nekhelesh Ramananthan]

I have added quicklist for totem which provides options for play,next,previous,mute and fullscreen.

[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

[Nekhelesh Ramananthan (nik90)]

I have updated the quicklist with the above mentioned changes which were required. Please review the latest code again. Thank you.

[Nekhelesh Ramananthan (nik90)]

I have reported this bug upstream with the patch.

[Sebastien Bacher (seb128)]

Thank you for your work, you need to prefix the Name with a "_" so they are translatable though

Could you also do a different patch for the keywords and the lists?

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

review: Needs Fixing

[Stéphane Graber (stgraber)]

Manually merged into lp:~ubuntu-desktop/totem/ubuntu as a Debian patch in debian/patches and uploaded.

=== modified file 'data/totem.desktop.in.in.in'
--- data/totem.desktop.in.in.in	2009-05-21 20:20:37 +0000
+++ data/totem.desktop.in.in.in	2012-03-05 16:43:47 +0000
@@ -14,3 +14,29 @@
 X-GNOME-Bugzilla-OtherBinaries=totem-video-indexer;totem-video-thumbnailer;totem-audio-preview;
 X-GNOME-Bugzilla-ExtraInfoScript=@FULL_LIBEXECDIR@/totem/totem-bugreport.py
 StartupNotify=true
+Actions=Play;Next;Previous;Mute;Fullscreen;
+
+[Desktop Action Play]
+_Name=Play/Pause
+Exec=totem --play-pause
+OnlyShowIn=Unity;
+
+[Desktop Action Next]
+_Name=Next
+Exec=totem --next
+OnlyShowIn=Unity;
+
+[Desktop Action Previous]
+_Name=Previous
+Exec=totem --previous
+OnlyShowIn=Unity;
+
+[Desktop Action Mute]
+_Name=Mute
+Exec=totem --mute
+OnlyShowIn=Unity;
+
+[Desktop Action Fullscreen]
+_Name=Fullscreen
+Exec=totem --fullscreen
+OnlyShowIn=Unity;