PapirusDevelopmentTeam / arc-kde

Arc KDE customization
https://git.io/arc-kde
GNU General Public License v3.0
901 stars 53 forks source link

Suggestion : Battery Icon #53

Closed zazatek closed 6 years ago

zazatek commented 6 years ago

Hi,

the theme is very very great, but I have some problem with the battery :/

It's very difficult to see if the pc is charging or not, can we do something ?

Without charging : https://vgy.me/2xAML3.png

With charging : https://vgy.me/hLGqxS.png

This is the only problem with this theme, we can't know the position of the battery.

And, do u think that we can do a red icon when the battery is critical ?

Thanks!

varlesh commented 6 years ago

Hi, sorry but we not use red color on plasma theme. For notification and critical messages used blue color #5294e2ff i found only 2 solution: 1- use blue color 2- add stroke 0.5 px 2017-08-20 07-17-02

zazatek commented 6 years ago

I think that the blue color is better for eyes. But the Stroke is more correlated with the theme. What do you think ?

varlesh commented 6 years ago

For Arc Color blue it's Highlight plasma color and maybe with combination Text color - this could be not good (with different color scheme)

varlesh commented 6 years ago

I think stroke variant better, because universal for Arc Dark and Arc Color too

zazatek commented 6 years ago

Ok nice, I am using Arc Dark.

Do you know how can I add the Stroke ? (I am on Fedora Plasma).

varlesh commented 6 years ago

Do you know how can I add the Stroke ? (I am on Fedora Plasma).

Not needed i'm fixed this soon on Master branch

zazatek commented 6 years ago

Ok thanks

varlesh commented 6 years ago

@zazatek Please check

varlesh commented 6 years ago

Final result not like me, i'm del stroke and colorize AcAdapter object to blue: https://github.com/PapirusDevelopmentTeam/arc-kde/commit/0009a466f06f37e1d6fb88ab6518b7d636e938cc 2017-08-20 08-54-43