papirus-icon-theme icon indicating copy to clipboard operation
papirus-icon-theme copied to clipboard

Wrong xfce4-power-manager tray icon color

Open aacebedo opened this issue 3 years ago • 16 comments

  • Distribution: Endeavour OS (arch based)
  • Desktop environment: Xfce + i3
  • Installing method: aur community/papirus-icon-theme 20210302-1
  • App name: xfce4-power-manager 4.16.0

I use the vimix-dark-laptop-ruby theme and the PapirusDark theme. The icon of the tray despite being white on my disk is recolored to a dark color, making it invisible. I checked with strace xfce4-power-manager that the icon loaded is correct /usr/share/icons/Papirus-Dark/symbolic/status/battery-level-80-symbolic.svg But it is displayed in a dark color: xfce4-power-manager

The only way to fix this is to add * {color:#FFFFFF;} to ~/.local/gtk-3.0/gtk.css however it has impact on other places than this specific icon

aacebedo avatar Mar 28 '21 11:03 aacebedo

https://github.com/PapirusDevelopmentTeam/papirus-icon-theme/issues/2484

varlesh avatar Mar 28 '21 12:03 varlesh

I don't think that's the same as #2484, the icon is correctly found and displayed but with the wrong styling. Do you think it is related to the gtk theme or the application instead of the icons theme ?

aacebedo avatar Mar 28 '21 12:03 aacebedo

Papirus - it's Darker theme, but symbolic icons need get color from GTK theme. This icon not get color from GTK Theme...

varlesh avatar Mar 28 '21 12:03 varlesh

@aacebedo what this orange icon on tray? And why wifi icon very small?

varlesh avatar Mar 28 '21 12:03 varlesh

what this orange icon on tray? And why wifi icon very small?

The orange icon is the seafile application (www.seafile.com). The wifi is small because I was missing a bar in reception quality (if you zoom you can see the darker background), so it's working correctly.

Ok so even if I see that the battery icon color is correct in Papirus-Dark, it's color shall have been changed by the theme to match the dark theme ? In this case I shall create an issue on the vmix theme.

aacebedo avatar Mar 28 '21 13:03 aacebedo

On Papirus Dark used only light symbolic icons, you can check it... But it's irrelevant, symbolic colors draw from GTK

varlesh avatar Mar 28 '21 13:03 varlesh

Ok thank you, I'll do an issue on the GTK theme, maybe there is something wrong with xfce4-power-manager.

aacebedo avatar Mar 28 '21 13:03 aacebedo

seafile icons used from binary qrc: https://github.com/haiwen/seafile-client/blob/master/src/ui/tray-icon.cpp#L492

varlesh avatar Mar 28 '21 13:03 varlesh

seafile icons used from binary qrc: https://github.com/haiwen/seafile-client/blob/master/src/ui/tray-icon.cpp#L492

You are right, I did not noticed the seafile icon because it was readable on my theme, They shall probably rewrite that to use a theme provided icon or at least a svg icon to let the theme recolorize it I guess.

aacebedo avatar Mar 28 '21 13:03 aacebedo

I'm now test manjaro xfce 4.16 and battery icon show correct... BUT i'm not have battery have only stationary computer). image image I think it's upstream xfce4-power-manager bug. Also only battery icons used symbolic style... All other tray elements not symbolic

varlesh avatar Mar 29 '21 06:03 varlesh

See last screen all icons used color #444444, but battery use from GTK Theme #323232

varlesh avatar Mar 29 '21 06:03 varlesh

What's your gtk theme ? I guess I shall be able to display then correct icon like you even if it is not the exact color. My version of xfce4-power-manager seems to be the same as you. Note I use polybar for the tray icon zone.

aacebedo avatar Mar 29 '21 06:03 aacebedo

Do you think the bug shall be reopened to keep track of this ?

aacebedo avatar Mar 29 '21 06:03 aacebedo

It's built in Manjaro XFCE Themes... I think better way for you - found another power manager applet for battery, with libappindicator for example

varlesh avatar Mar 29 '21 07:03 varlesh

https://github.com/maateen/battery-monitor

varlesh avatar Mar 29 '21 07:03 varlesh

https://github.com/maateen/battery-monitor

Thank you for the info but I prefer not to use unmaintained projects so I'll stick with the xfce4-power-manager if there is nothing else for moment. I hope the bug will be fixed. I notified vimix theme project of the issue. Maybe they'll find then correct css selector to add to target only this application.

aacebedo avatar Mar 29 '21 07:03 aacebedo

I managed to customize this color: you can use

#xfce4-power-manager-plugin image {
  -gtk-icon-palette: warning #ff0000, success #00ff00, error #0000ff;
  color: #ffff00;
}

Of course you can change your colors here. color is for "normal" status, warning is for "caution", success is for full/charging, and error is for empty battery.

renyhp avatar Jan 05 '23 22:01 renyhp

I managed to customize this color: you can use

#xfce4-power-manager-plugin image {
  -gtk-icon-palette: warning #ff0000, success #00ff00, error #0000ff;
  color: #ffff00;
}

Of course you can change your colors here. color is for "normal" status, warning is for "caution", success is for full/charging, and error is for empty battery.

I'm pretty new with this, but can you tell me where to change/add this? Adding it to ~/.config/gtk-3.0/gtk.css and restarting the power-panel changes nothing...

MrHappy avatar Jan 09 '23 06:01 MrHappy