xpra icon
Bug tracker and wiki

This bug tracker and wiki are being discontinued
please use https://github.com/Xpra-org/xpra instead.

Opened 8 years ago

Closed 2 years ago

Last modified 3 months ago

#406 closed enhancement (fixed)

fix appindicator so we can use it for forwarding system trays

Reported by: Antoine Martin Owned by: Antoine Martin
Priority: minor Milestone: 2.5
Component: client Version:
Keywords: ubuntu Cc:

Description (last modified by Antoine Martin)

I'm not even sure this is possible, but since Ubuntu does not support the regular StatusIcon correctly anymore (and with newer versions, it simply does not show up at all), better have something - even something ugly.

And this is going to be ugly. Their API is an ugly joke:

First, just like with OSX (#404), we don't know the location of the appindicator until it is used, and even then finding the exact coordinates is a complete nightmare (see https://bugs.launchpad.net/screenlets/+bug/522152/comments/11)

Then we can't just catch the events we want (hover, clicks), we only get the option of having a menu (how convenient and modern - NOT!), so we will have to populate this menu with something - maybe have one entry for each action (left click, right click)... but we don't even know which clicks do anything at the other end.

Can someone from Ubuntu please fix this - it's their horrible mess.


Change History (10)

comment:1 Changed 8 years ago by Antoine Martin

Description: modified (diff)

See also #636

Last edited 6 years ago by Antoine Martin (previous) (diff)

comment:2 Changed 3 years ago by Antoine Martin

Resolution: wontfix
Status: newclosed

That thing is best forgotten.

Most of the Ubuntu wiki pages lead to 404s.
It never worked properly, and is now causing crashes under 18.04 with unity.

python3 -c "from gi.repository import AppIndicator3;AppIndicator3.Indicator()"


Last edited 3 years ago by Antoine Martin (previous) (diff)

comment:3 Changed 2 years ago by Antoine Martin

See also #2179 / #1818

Last edited 2 years ago by Antoine Martin (previous) (diff)

comment:4 Changed 2 years ago by Antoine Martin

Resolution: wontfix
Status: closedreopened

Needed because statusicon is now even more broken..

comment:5 Changed 2 years ago by Antoine Martin

Resolution: fixed
Status: reopenedclosed

Fixes in:

  • r21893 packaging: recommend appindicator and top-icons-plus extensions so we can show something somewhere - hopefully
  • r21894: use appindicator on "unity" and "gnome" DEs
  • r21898: fix appindicator3 (was invalid argument causing the segfault..), enable it for kde
  • r21982: better DE name matching
  • r21983 + r21984: debian package dependencies (see python-appindicator removed in buster..)
Version 0, edited 2 years ago by Antoine Martin (next)

comment:6 Changed 2 years ago by Antoine Martin

Will follow up in #2197

comment:7 Changed 2 years ago by Antoine Martin

More fixes:

  • r22004 + r22007 don't use /tmp for temporary icon files we have to use with appindicator (..), move them to /run/user/$UID/xpra/tmp
  • r22005 keep the file around until we have the next one (async code in appindicator was making this racy)
  • r22006 better debugging
  • r22008 fix for python3 server icon change, which broke theme icons
  • r22009 RPM dep update

comment:8 Changed 21 months ago by Antoine Martin

Related tickets: #2161, #2197, #2242, #476

comment:9 Changed 14 months ago by Antoine Martin

Milestone: future2.5

comment:10 Changed 3 months ago by migration script

this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/406

Note: See TracTickets for help on using tickets.