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

[Nekhelesh Ramananthan]

Remmina Quicklist support added

[Sebastien Bacher (seb128)]

Thank you for your work, some comments:

- could you send that upstream as well and add the bug reference here
- those items need to use the standard menus style, i.e "Create New Connection Profile" and "Start Remmina Minimized"

review: Needs Fixing

[Michael Hall (mhall119)]

Thanks so much for your submission!

Unfortunately your submission is for a package that can only be translated by the upstream developers, so you will need to send your changes to them first. If they are accepted by the upstream project, they will then be included in the next release of Ubuntu.

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 make these changes and submit them to the upstream projects so they can be included in future versions of project and future releases of Ubuntu.

[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

[lp:~nik90/ubuntu/precise/remmina/add_quicklist]

18. By Nekhelesh Ramananthan <email address hidden> on 2012-02-29

Updated quicklist according to latest specifications

[Nekhelesh Ramananthan (nik90)]

@Michael Hall, I visited their website where they mention that translations are now done in Launchpad which are imported before a new release. You can read it here http://remmina.sourceforge.net/contact.shtml

Also I visited their forums which you can see for yourself does not have any recent activity at all. Infact the last post in the development forum was on the 25th Jan 2012. What do I do? I am unable to find where their code is hosted.

[Michael Hall (mhall119)]

Ah, you are right, Remmina has been moved to the Main repository for 12.04, so you should be all set with these changes.

Thanks again.

[Nekhelesh Ramananthan (nik90)]

I like someone to confirm this. I found the upstream hosted at github (https://github.com/FreeRDP/Remmina). I added a merge request which has been immediately accepted. Here is the merge history for remmina. https://github.com/FreeRDP/Remmina/commits/master

Is this sufficient or am I doing something wrong here?

[Sebastien Bacher (seb128)]

That looks great Nekhelesh, thanks for your work! We will make sure that's shipped in precise. One small comment, the OnlyShowIn should have have a trailing ";", i.e "OnlyShowIn=Unity;", the current way will probably work in practice but will trigger warning about the syntax when given to parser tools

[Sebastien Bacher (seb128)]

(just noticed that the wiki instructions have that mistake, I will get that fixed as well)

[Michael Hall (mhall119)]

Wiki has been updated

[lp:~nik90/ubuntu/precise/remmina/add_quicklist]

19. By Nekhelesh Ramananthan <email address hidden> on 2012-03-01

Correct mistake in quicklists

[Nekhelesh Ramananthan (nik90)]

I have made the changes and have also uploaded them upstream.

[Sebastien Bacher (seb128)]

Thanks, uploaded to precise. Since you have a bug upstream could you maybe also open a bug to point them that their .desktop is not translatable?

=== modified file 'remmina/desktop/remmina.desktop'
--- remmina/desktop/remmina.desktop	2012-02-15 13:27:09 +0000
+++ remmina/desktop/remmina.desktop	2012-03-01 00:29:18 +0000
@@ -9,4 +9,15 @@
 Type=Application
 Actions=
 Categories=GTK;GNOME;X-GNOME-NetworkSettings;Network;
+Actions=Profile;Tray;
+
+[Desktop Action Profile]
+Name=Create a New Connection Profile
+Exec=remmina --new
+OnlyShowIn=Unity;
+
+[Desktop Action Tray]
+Name=Start Remmina Minimized
+Exec=remmina --icon
+OnlyShowIn=Unity;