Skip to content
This repository has been archived by the owner on Oct 29, 2018. It is now read-only.

Tray icon not visible in Mint Cinnamon 17.2 #192

Closed
souvikonline opened this issue Aug 4, 2015 · 14 comments · Fixed by #340
Closed

Tray icon not visible in Mint Cinnamon 17.2 #192

souvikonline opened this issue Aug 4, 2015 · 14 comments · Fixed by #340

Comments

@souvikonline
Copy link

I am using the Void (http://cinnamon-spices.linuxmint.com/uploads/themes/C7AR-WYUC-R6K0.png)
screenshot from 2015-08-04 10 35 34
theme for Cinnamon in Linux Mint 17.2. The tray icon is not visible. Please see the screenshot.

@gmorten1
Copy link

gmorten1 commented Aug 4, 2015

i have the same or similar problem in ubuntu 14.04 (trusty) with the Mate desktop (gnome-based desktop similar to cinnamon). i have the latest scudcloud 1.0-53 from the repo. i didn't have this problem with the previous version

specifically, the panel icon disappears when i have a pending notification. i see the normal empty cloud icon when nothing is pending,
screenshot from 2015-08-04 08 56 33

but when a DM comes in the icon first flashes the blue cloud icon (no number) and then disappears completely except for a black underline
screenshot from 2015-08-04 08 58 38

the icon files (e.g. scudcloud-attention-1.svg) look ok if i open them in an external viewer

@gmorten1
Copy link

gmorten1 commented Aug 4, 2015

incidentally, looking at souvik's screenshot i think he has a different problem (which i also encountered before) -- the pale color of the transparent "empty cloud" icon scudcloud.svg is almost an exact match for the default panel background color. i "fixed" this by changing my panel background.

@0xd3adsh3ll
Copy link

_003
_002

@gmorten1 I have the same problem as yours.
linux mint 17.2

@roman-kiselenko
Copy link
Contributor

same
other

  • +1 Ubuntu 14.04. Gnome fallback
    No icon for a new message

@sergey-sukiyazov
Copy link

Hi
I found cause of this problem. It happens for me under Ubuntu 14.04 LTS too.
Problem is in file systray.py at line 38:

    def setCounter(self, i):
        print( "i =", i)
        print( "scudcloud-attention-"+str(int(i)))
        if 0 == i:
            if True == self.urgent:
                self.setIcon(QtGui.QIcon.fromTheme("scudcloud-attention"))
            else:
                self.setIcon(QtGui.QIcon.fromTheme("scudcloud"))
        elif i > 0 and i < 10:
            self.setIcon(QtGui.QIcon.fromTheme("scudcloud-attention-"+str(i)))
        elif i > 9:
            self.setIcon(QtGui.QIcon.fromTheme("scudcloud-attention-9-plus"))

there is code

self.setIcon(QtGui.QIcon.fromTheme("scudcloud-attention-"+str(i)))

but value of i became as float, i.e. we got 2.0 instead of 2 and we have wrong name for icon. So, we need to convert float to integer before call str(...), i.e.:

self.setIcon(QtGui.QIcon.fromTheme("scudcloud-attention-"+str(int(i))))

this solve the problem.

raelgc added a commit that referenced this issue Aug 6, 2015
@raelgc
Copy link
Owner

raelgc commented Aug 6, 2015

Thank you for the investigation and fix @fishbullet !

@gmorten1
Copy link

gmorten1 commented Aug 6, 2015

awesome guys, just updated to 1.0-55 and my issue is fixed. thanks!
image

as i mentioned previously, i now think that the original issue reported by souvik is a different problem -- the "empty cloud" icon in the hicolor icon theme doesn't have enough contrast on the light gray default panel background color on some gnome-based desktops. if you zoom on his original screenshot it's evident that the icon is actually there, but it's nearly invisible due to lack of contrast.

@roman-kiselenko
Copy link
Contributor

Thanks @sergey-sukiyazov for the investigation, i just make a fast PR.

@raelgc
Copy link
Owner

raelgc commented Aug 6, 2015

Well, nice team work :)

@msaggiorato
Copy link

Hey guys, i'm getting this in MATE too, not sure if this was already pushed to the repository that we use to install the app.

Also, it would be great to be able to change the tray icon like it's possible to change the app icon.

@grinderX19
Copy link

Confirming the same for Xfce4.

@raelgc
Copy link
Owner

raelgc commented Sep 23, 2015

@msaggiorato But you can do the same as the for app icon. Just remember to use dpkg-divert to keep your changes between updates.

For some reason, under Xfce4, Qt is using the icon for dark themes, located at /usr/share/icons/ubuntu-mono-dark/scalable/apps/scudcloud.svg and should be using /usr/share/icons/ubuntu-mono-light/scalable/apps/scudcloud.svg.

If someone play with the code under Xfce4, the file is systray.py and this is the line:

 self.setIcon(QtGui.QIcon.fromTheme("scudcloud"))

@pescobar
Copy link

I am getting the same problem in Linux Mint 17.2 with MATE using scudcloud 1.1-1. I have checked that the patch in systray.py which is mentioned in previous comments is applied.

When opening scudcloud from the shell I get this message which I don't know if could be related:

"sni-qt/6099" WARN  10:46:34.586 void StatusNotifierItemFactory::connectToSnw() Invalid interface to SNW_SERVICE 
No bp log location saved, using default.
[000:000] Cpu: 6.69.1, x4, 2600Mhz, 7927MB
[000:000] Computer model: Not available
[000:000] Browser XEmbed support present: 1
[000:000] Browser toolkit is not Gtk2 (0).
[000:000] Using Xt toolkit
No bp log location saved, using default.
[000:000] Cpu: 6.69.1, x4, 2600Mhz, 7927MB
[000:000] Computer model: Not available
java version "1.7.0_85"
OpenJDK Runtime Environment (IcedTea 2.6.1) (7u85-2.6.1-5ubuntu0.14.04.1)
OpenJDK 64-Bit Server VM (build 24.85-b03, mixed mode)

UPDATE: When I get a notification I can see something:
systray

maybe the default icon is not visible with my theme?

@pescobar
Copy link

after reading this issue #229 I found a workaround doing this:

sudo cp /usr/share/icons/hicolor/scalable/apps/scudcloud.svg /usr/share/icons/hicolor/scalable/apps/scudcloud.svg.backup

sudo cp /usr/share/icons/ubuntu-mono-light/scalable/apps/scudcloud.svg /usr/share/icons/hicolor/scalable/apps/scudcloud.svg

Now it's not perfect but at least I see something in the systray:

systray2

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

Successfully merging a pull request may close this issue.

9 participants