indicator for deluge

Some relevant upstream discussion starting here:

http://<email address hidden>/msg00794.html

I was going to file a bug about it, I hope they implement this for maverick.

I'm working on this, but work is being blocked by bug #608219.

The attached patch contains the bulk of the code for creating an Application Indicator for Deluge HEAD.

It doesn't work yet, as it depends on bug #608219, and might need some changes after that bug is fixed.

By default the Application Indicator component is disabled and the systemtray extension is used. Set the preference 'enable_application_indicator' to True. You can find it at 'deluge/ui/gtkui/gtkui.py:113'.

I'd like to include this upstream, but I am not entirely convinced of the approach taken in this patch. Adding an additional component that is essentially a copy and paste of the existing SystemTray component isn't really ideal. There is way too much code duplication and this will become a maintenance nightmare.

The better approach would be to modify SystemTray to use either a gtkStatusIcon or the Appindicator possibly based on a configuration value or simply in an order of preference, eg, use the Appindicator if the 'import appindicator' succeeds. I haven't taken a hard look at the patch, but it's likely that this could be accomplished with far less new lines of code. There are other options too, you could for instance, create a base class and subclass the SystemTray and Appindicator respectively.

Thanks for your time in adding this functionality, I hope that we can add it to Deluge in the future.

Thats really not too hard to do, so I'll have a poke at it tomorrow as I've just been given this bug :)

Are there any news on indicator support for deluge? It would be ver nice if this was included.

There is some large work planned to merge the GtkMenu processing code of AppMenu and Application Indicators, which will probably solve a lot of bugs we are experiencing right now, and which are also blocking this indicator.

I have already got an indicator that is almost finished, it is just buggy. When the bugs are gone I expect it to be very easy to finish it and submit the Deluge Application Indicator.

latest patch, this almost works around the download/upload speed menu items not having the submenus but still those submenus emit signals on the wrong radio menu item.

This patch also merges together the systemtray and appindicator modules as they were far too similar.

This bug is related to; https://bugs.launchpad.net/indicator-application/+bug/608219

@Andrew Resch: Is the patch in comment #10 more to your liking, would you care to accept this upstream?

This is already in 1.3-stable branch http://dev.deluge-torrent.org/ticket/1514#comment:3, but deb is not updated since patch inclusion so i cant say if it works or not

It should be included in the next Deluge point release 1.3.2. Here's the commit:

http://git.deluge-torrent.org/deluge/commit/?h=1.3-stable&id=fd3a33af03b96a868545fa20e9f67af7d1a1b157

I'll look into backporting this to Ubuntu.

This bug was fixed in the package deluge - 1.3.1-1ubuntu2

---------------
deluge (1.3.1-1ubuntu2) natty; urgency=low

  * Backport indicator support from upstream 1.3.x
    stable branch (LP: #584669).
 -- Andrew Starr-Bochicchio <email address hidden> Sun, 13 Mar 2011 14:36:24 -0400