Bug 424283 - Split applet into two: "Power Management" and "Brightness"
Summary: Split applet into two: "Power Management" and "Brightness"
Status: CONFIRMED
Alias: None
Product: plasmashell
Classification: Unclassified
Component: Battery Monitor (show other bugs)
Version: master
Platform: Other Linux
: LO wishlist
Target Milestone: 1.0
Assignee: Kai Uwe Broulik
URL:
Keywords: usability
Depends on:
Blocks: 448271
  Show dependency treegraph
 
Reported: 2020-07-16 11:38 UTC by medin
Modified: 2022-01-11 17:10 UTC (History)
6 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description medin 2020-07-16 11:38:25 UTC
It's not logical/intuitive to click on icon with battery symbol to change the brightness ! The icon should be changed to "sun" symbol and be like what we see on sound icon, and the battery settings can stay embedded inside the widget.
Comment 1 David Edmundson 2020-07-16 11:50:03 UTC
Whilst I agree with your comment about it being un-intuitive the proposal has severe drawbacks.

>The icon should be changed to "sun" symbol

Then we're not showing the battery information in an easily visible place. 
This is important information to surface visually.
Comment 2 medin 2020-07-16 11:57:42 UTC
(In reply to David Edmundson from comment #1)
> Whilst I agree with your comment about it being un-intuitive the proposal
> has severe drawbacks.
> 
> >The icon should be changed to "sun" symbol
> 
> Then we're not showing the battery information in an easily visible place. 
> This is important information to surface visually.

For me, It's not logical to always link brightness control with power settings. I can have max brightness with battery mode enabled, and I can have min brightness with AC mode. The only link is the impact of "Save mode" on brightness level which is valid only when battery is used but for desktops or laptops using always AC mode it's not valid at all.
Comment 3 medin 2020-07-16 12:02:28 UTC
(In reply to David Edmundson from comment #1)
> Whilst I agree with your comment about it being un-intuitive the proposal
> has severe drawbacks.
> 
> >The icon should be changed to "sun" symbol
> 
> Then we're not showing the battery information in an easily visible place. 
> This is important information to surface visually.

The brightness icon should be separated from battery settings, so the brightness icon is always visible with or without battery used, but the battery icon should be enable only when a battery is detected and shows only battery level status.

The power settings/save mode can always be accessed via global settings.
Comment 4 Nate Graham 2020-07-17 03:04:34 UTC
There's some logic to the idea of separating brightness settings from battery settings. All systems will have brightness control, while not all have any batteries or battery-powered devices connected.

We need a place for laptop users to be able to easily see their battery status, and there are plans afoot to add power profiles to that applet too. So we can't just delete the battery applet and make all of its settings only accessible in System settings.

Thus, we would need two applets: one for screen/keyboard brightness, and another one for battery and power management stuff. Doing this would result in people with laptops and battery-powered connected devices gaining an additional System Tray icon. I guess that's not a serious problem, but some people really don't like having tons and tons of items in their System Tray. Also, when you want to save power, reducing the brightness is one of the best ways to do it, so it makes a bit of sense to have brightness be in the power management applet.

Undecided. Perhaps the icon itself could somehow be changed to visually incorporate brightness as well?
Comment 5 Nowshed H. Imran 2020-11-19 16:36:42 UTC
I would like to propose a solution. It is ok not separating the icons. 
However, as soon as the AC power is connected, the battery icon gets hidden in status and notification menu. 
So, if I need to change the brightness, I can't do it with a single tap.
Solution: Battery icon is always visible in taskbar.
Comment 6 Nate Graham 2022-01-04 17:45:44 UTC
Now that Bug 415073 is fixed, this is now only relevant for machines with batteries.

The only real fix I can think of is to split the applet's content into two new ones:

Power Management
- Battery information
- Sleep & Screen locking blocking checkbox
- Power profiles

Brightness
- Screen brightness
- Keyboard brightness


Not sure how desirable that would be though. Would need more discussion. Lowering priority.
Comment 7 Nowshed H. Imran 2022-01-07 11:16:02 UTC
That would be the perfect solution. Splitting these two would make these two options more modular.
Comment 8 Photon 2022-01-10 06:31:32 UTC
Hey, what about adding nightlight controls (warmer, cooler) in the brightness applet? Just like this concept: https://www.reddit.com/r/kde/comments/rzvw1u/redesign_concepts_and_ideas_for_plasma_yay_or_nay/
Comment 9 Nowshed H. Imran 2022-01-10 11:54:45 UTC
This might work.
Comment 10 Nate Graham 2022-01-10 15:11:33 UTC
That's a good idea IMO, but unrelated to this bug report; can you file a new one requesting that?
Comment 11 Nowshed H. Imran 2022-01-11 17:05:54 UTC
Repoted here:

https://bugs.kde.org/show_bug.cgi?id=448271