xpra icon
Bug tracker and wiki

Opened 6 years ago

Closed 3 months ago

Last modified 2 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: future
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.

Links:

Change History (7)

comment:1 Changed 6 years ago by Antoine Martin

Description: modified (diff)

See also #636

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

comment:2 Changed 7 months 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()"

segfaults..

Last edited 7 months ago by Antoine Martin (previous) (diff)

comment:3 Changed 3 months ago by Antoine Martin

See also #2179 / #1818

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

comment:4 Changed 3 months ago by Antoine Martin

Resolution: wontfix
Status: closedreopened

Needed because statusicon is now even more broken..

comment:5 Changed 3 months ago by Antoine Martin

Resolution: fixed
Status: reopenedclosed

Fixes in:

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

comment:6 Changed 2 months ago by Antoine Martin

Will follow up in #2197

comment:7 Changed 2 months 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
Note: See TracTickets for help on using tickets.