search

alex2108 / syncthing-tray

trayicon to display status of syncthing
MIT License
57 stars 9 forks source link

the icon doesn't change in tint2 #16

Open graboluk opened 8 years ago

graboluk commented 8 years ago

I run syncthing-tray with ./syncthing-tray -target="http://localhost:8384"

As expected, in my tint2 panel a red icon appears, and almost immediately it turns blue. However, there are no further changes in the icon's state, although in the console I get a lot of messages like this: 2015/09/29 12:28:06 main.go:345: updating status 2015/09/29 12:28:06 main.go:375: connected 3 2015/09/29 12:28:06 main.go:390: ul+dl 2015/09/29 12:28:06 trayhost.go:113: Setting icon /tmp/trayhosticon100407285 (id: 8) 2015/09/29 12:28:07 main.go:345: updating status 2015/09/29 12:28:07 main.go:375: connected 3 2015/09/29 12:28:07 main.go:390: ul+dl

(and the icon /tmp/trayhosticon100407285 is correct).

alex2108 commented 8 years ago

Not sure if I can do something about that. Sounds like a bug in trayhost or tint2 similar to the bug I see in ubuntu, there the icon is updated but the text is not updated any more even if my program definately sends to command to update it.

graboluk commented 8 years ago

I see. For whatever it's worth, I think it's not tint2 since dropbox and wicd-gtk icons work fine. I use openbox without gnome/kde/xfce/etc. so if trayhost is doing something fancy, instead of the freedesktop system tray protocol, then indeed trayhost could be at fault (still I'm puzzled that the icon is red at the start, and then it does change to blue, so maybe it's something else altogether...)

And BTW - you say "text is not updated", does it mean there should be a tooltip? I don't have any tooltips (and again dropbox and wicd-gtk have correct tooltips).

alex2108 commented 8 years ago

there should be a menu when right clicking, this is what does not always update on ubuntu. Mouseover at least exists for me on windows (just says syncthing-tray, so nothing important if it is not there), Ubuntu also does not show that, but the other icons I have there also don't show anything so maybe that is a Ubuntu thing.

I just see that setIcon can return an error, looks like I ignored that... You can try this release to see if it prints something other than after the state (ul+dl, idle etc) or if it works since I updated trayhost even if the few commits sound like bugfixes for OSX and not for Linux (also a lot of other things changed here and stuff might be broken, I'm in the middle of changing things there, but for a test this should not matter) https://www.dropbox.com/s/rbq7x26kyutoocj/syncthing-tray?dl=0 (64bit version, hope you don't need 32).

graboluk commented 8 years ago

I actually need 32 :-)

alex2108 commented 8 years ago

ok next try: https://www.dropbox.com/s/5k7hblehebtkj6n/syncthing-tray-linux32.tar.gz?dl=0

graboluk commented 8 years ago

After first 2015/09/29 15:04:25 trayhost.go:116: Setting icon /tmp/trayhosticon234072791 (id: 6) the icon became empty and stayed this way (also after gettin back to idle)

alex2108 commented 8 years ago

another try: https://www.dropbox.com/s/5k7hblehebtkj6n/syncthing-tray-linux32.tar.gz?dl=0 empty icon sounds like the file does not exist, try something like this with the new version (path has to exists, otherwise it will crash) ./syncthing-tray -icondir="/home/alex/trayhost-icons"

If that also does not work I have no idea what is going on there...

graboluk commented 8 years ago

It doesn't work on my main computer. However it works on two other machines which also have tint2 (it's possible that the old version would also work on these two computers).

In case it's of relevance, my main computer is a bit old and doesn't have much more than pure openbox (e.g. it doesn't have any qt libs etc). And in any case thanks for trying!