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

New classes #2448

Closed
varlesh opened this issue Nov 26, 2020 · 8 comments
Closed

New classes #2448

varlesh opened this issue Nov 26, 2020 · 8 comments

Comments

@varlesh
Copy link
Contributor

varlesh commented Nov 26, 2020

Ok guys. We support more De's and earlier adhered to the concept monochrome tray icons wigh 2 colors:

  • ColorScheme-Text
  • ColorScheme-Highlight
    image
    But now it's not enough and we are at a loss.
    Our main DE's - GNOME (GTK) & KDE (Qt) but the have differen classes for staus:
    GNOME have classes:
  • warning (Yellow/Orange)
  • error (RED)
  • success (GREEN)

KDE have classes:

  • ColorScheme-NegativeText (RED)
  • ColorScheme-PositiveText (GREEN)
  • ColorScheme-NeutralText (YELLOW/ORANGE)
  • ColorScheme-Highlight (BLUE)

As you see we have only one problem use succes status as ColorScheme-PositiveText or ColorScheme-Highlight.
Yes... yes... we lost monochrome accent color, but can have more variation for status.
OK. Now we dispute about Battery icon and we can't decide what to do:
Now we have this:

image

Future @SmartFinn version:

image

Future @varlesh version:
image

@varlesh
Copy link
Contributor Author

varlesh commented Dec 4, 2020

@uvulpos Battery updated on master branch

@mcnesium
Copy link

mcnesium commented Feb 8, 2021

Looks like class="ColorScheme-PositiveText" is now the default in all the battery icons. As far as I understood, this is the KDE-way of doing things. However, it breaks monochrome designs. For the state-ok icon, this has already been reverted to ColorScheme-Text to default to monochrome view. Is there a chance that this will happen to the battery icons, as well?

@varlesh
Copy link
Contributor Author

varlesh commented Feb 9, 2021

@mcnesium Please see #2447.

  • Monochrome (normal status) 30-100% without AC & 100% with AC. We specially doing this for not distract user, when you connect AC with full battery.
  • Green (positive status) icons use when you charging from 0% - 90%
  • Yellow (warning status) battery 20%
  • Red (critical status) battery 0-10%

@varlesh
Copy link
Contributor Author

varlesh commented Feb 9, 2021

AC disconnected:
image
AC connected:
image

@uvulpos
Copy link

uvulpos commented Feb 9, 2021

Looks great! Good job! 😄

@ozls
Copy link

ozls commented Feb 27, 2021

I don't understand why this version eventually won. It's beautiful, but I wholeheartedly agree with the input of @SmartFinn on bug #2447. My everyday situation is a laptop running on AC with a half empty battery that is not charging (for battery longevity reasons), and I find it very distracting to have the green accent in this case. On the other hand, in a situation where I'm running solely on battery, I would find it very useful to be reminded of its state constantly. So what gives ? Is there a practical reason behind this choice ? Can I do something ?

@intrnl
Copy link

intrnl commented Mar 20, 2021

Was wondering the same, I find the green battery to be distracting.
image

@95440b97d
Copy link

Can this be reopened? I agree with @ozls. The version of @SmartFinn makes absolutely more sense.

Keeping the battery not fully charged (on purpose) and plugged in nags you with an green accent color where attention is not needed (we are plugged in after all).

Whereas when the battery is discharging there's no nagging accent color (the color is monochrome). Here a nagging color is actually beneficial to remind the user of something he needs to pay attention to.

Using stock Gnome desktop.

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

No branches or pull requests

7 participants