Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix appindicator so we can use it for forwarding system trays #406

Closed
totaam opened this issue Aug 8, 2013 · 8 comments
Closed

fix appindicator so we can use it for forwarding system trays #406

totaam opened this issue Aug 8, 2013 · 8 comments

Comments

@totaam
Copy link
Collaborator

totaam commented Aug 8, 2013

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:

@totaam
Copy link
Collaborator Author

totaam commented Aug 8, 2013

See also #636

@totaam
Copy link
Collaborator Author

totaam commented Oct 8, 2018

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..

@totaam
Copy link
Collaborator Author

totaam commented Feb 26, 2019

See also #2179 / #1818

@totaam
Copy link
Collaborator Author

totaam commented Mar 6, 2019

Needed because statusicon is now even more broken..

@totaam
Copy link
Collaborator Author

totaam commented Mar 6, 2019

Fixes in:

@totaam totaam closed this as completed Mar 6, 2019
@totaam
Copy link
Collaborator Author

totaam commented Mar 7, 2019

Will follow up in #2197

@totaam
Copy link
Collaborator Author

totaam commented Mar 8, 2019

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

@totaam
Copy link
Collaborator Author

totaam commented Jul 24, 2019

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant