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

Light-coloured symbolic icon for power manager (attached). #158

Open
Thyname opened this issue Jun 22, 2019 · 14 comments
Open

Light-coloured symbolic icon for power manager (attached). #158

Thyname opened this issue Jun 22, 2019 · 14 comments

Comments

@Thyname
Copy link

Thyname commented Jun 22, 2019

I changed the colour to eeeeeeff so it can be used in elementary-xfce-dark panel icons.
ac-adapter-symbolic.zip

I don't know if it's useful without the other power manager icons (like battery levels, ups etc.), but posting just in case.

@m4u9
Copy link
Member

m4u9 commented Jun 24, 2019

I'm pretty sure that it already changes the color automatically when using a dark panel
see: https://en.opensuse.org/File:Screenshot_Leap-15.1-Xfce_Desktop.png

@Thyname
Copy link
Author

Thyname commented Jun 24, 2019

Well then there must be a light-coloured icon in your elementary-xfce-dark icon theme. Here in Fedora 30 it was using a hicolor type icon.
I admit I didn't check here upstream before opening this report (haven't checked yet).

Thanks.

@m4u9
Copy link
Member

m4u9 commented Jun 24, 2019

Yes I'm using the latest version from here (git master). Also the color of symbolic icons should be correctly assigned by the gtk theme so that when switching from dark to light you don't need two sets of icons. Maybe try changing you gtk theme or try the version from git here?

@Thyname
Copy link
Author

Thyname commented Jun 24, 2019

Ok, I'm not an expert, but you mean even if there's only a black symbolic icon gtk would be able to re-colourize it?
Because here elementary-xfce-dark/panel/48/ac-adapter.svg ( https://github.com/shimmerproject/elementary-xfce/blob/master/elementary-xfce-dark/panel/48/ac-adapter.svg ) links here https://github.com/shimmerproject/elementary-xfce/blob/master/elementary-xfce/status/48/battery-ac-adapter.svg

I'll check my panel with elementary-xfce and elementary-xfce-dark in a while.

@m4u9
Copy link
Member

m4u9 commented Jun 24, 2019

Ok, I'm not an expert, but you mean even if there's only a black symbolic icon gtk would be able to re-colourize it?

yes

Because here elementary-xfce-dark/panel/48/ac-adapter.svg ( https://github.com/shimmerproject/elementary-xfce/blob/master/elementary-xfce-dark/panel/48/ac-adapter.svg ) links here https://github.com/shimmerproject/elementary-xfce/blob/master/elementary-xfce/status/48/battery-ac-adapter.svg

It should read it from here: https://github.com/shimmerproject/elementary-xfce/blob/master/elementary-xfce/devices/symbolic/ac-adapter-symbolic.svg

I'll check my panel with elementary-xfce and elementary-xfce-dark in a while.

@ochosi
Copy link
Member

ochosi commented Jun 24, 2019

Colored icons are used in the trayicon and notification bubbles, symbolic icons are used in the xfpm panel plugin.

@Thyname
Copy link
Author

Thyname commented Jun 24, 2019

Right, but here it's like this:
elementary icon theme (note black symbolic icon also):
https://pasteboard.co/IkTve9I.png

elementary-xfce (symbolic black icon):
https://pasteboard.co/IkTsKMd.png

elementary-xfce-dark (coloured icon):
https://pasteboard.co/IkTwcG2.png

So unless I create a (light-coloured) symbolic icon in elementary-xfce-dark/panel/22 and 24 it loads this coloured icon.

@Thyname
Copy link
Author

Thyname commented Jun 24, 2019

Read your comment now ochosi, will check the panel plugin.
I'm referring to the tray icon.

@m4u9
Copy link
Member

m4u9 commented Jun 24, 2019

Read your comment now ochosi, will check the panel plugin.
I'm referring to the tray icon.

and what about the panel plugin?

@Thyname
Copy link
Author

Thyname commented Jun 24, 2019

Panel plugin works ok, just like yours. So for tray icon, the report is valid and theme needs these light-coloured symbolic icons in panel/22 and panel/24 in elementary-xfce-dark

@m4u9
Copy link
Member

m4u9 commented Jun 24, 2019

Panel plugin works ok, just like yours. So for tray icon, the report is valid and theme needs these light-coloured symbolic icons in panel/22 and panel/24 in elementary-xfce-dark

Status-notifier plugin has an option to force the use of symbolic icons, you could also try with that. Not sure that the systray aka notification-area has such option.

@ochosi probably something to bring up at xfce but I've been wondering whether it wouln't be better to make status-notifier and systray use symbolic icons by default so that they are consistent with the panel plugins. Doable?

@Thyname
Copy link
Author

Thyname commented Jun 24, 2019

And as you said, checking now with various icon themes, the icon colour adapts to the gtk theme for the plugin, but not for the tray icon (which needs a light-coloured icon for it, unless it's decided to keep the coloured icon as is).

@Thyname
Copy link
Author

Thyname commented Jun 24, 2019

Just to notice, I'm ok with whatever design decision, especially since the plugin uses the symbolic icon.

What actually triggered this icon issue with me was the fact that current 4.13 xfce-panel is very broken regarding icons when compositing is disabled, particularly in the tray, but also for example in applications menu (which can sometimes not refresh the icons).

I started thinking things were really broken overall in xfce 4.13 regarding icons. But it's basically the panel without compositing. With compositing turned on things appear to work as expected.

@ochosi
Copy link
Member

ochosi commented Jun 24, 2019

The systray currently doesn't support symbolic icons, although I have a branch that tries to implement that (based on what Mate and Cinnamon do). statusnotifier is not part of the default layout yet, but I'll add it

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

3 participants