shimmerproject / elementary-xfce

Elementary icons forked from upstream, extended and maintained for Xfce
GNU General Public License v2.0
274 stars 37 forks source link

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

Open Thyname opened 5 years ago

Thyname commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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

Thyname commented 5 years ago

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 commented 5 years ago

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

m4u9 commented 5 years ago

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

and what about the panel plugin?

Thyname commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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