Bug 442906 - Info duplication when a battery's health is low
Summary: Info duplication when a battery's health is low
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Power and Battery widget (show other bugs)
Version: 5.22.5
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Kai Uwe Broulik
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-09-24 17:53 UTC by ratijas
Modified: 2022-01-04 17:55 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.24
Sentry Crash Report:


Attachments
screenshot of battery health applet (57.55 KB, image/png)
2021-09-24 17:53 UTC, ratijas
Details

Note You need to log in before you can comment on or make changes to this bug.
Description ratijas 2021-09-24 17:53:35 UTC
Created attachment 141878 [details]
screenshot of battery health applet

SUMMARY
Two problems about this applet's visual design:

1. Information about battery health is effectively duplicated when battery's health is low.
2. Label "Battery health:" is too far from the numeric value on the other side of an applet. It's even worse when low health notice is on top, making it look misaligned and wrong.

STEPS TO REPRODUCE
1. Murder ur battery almost to death.
2. Or "just" buy an ASUS ROG laptop, and use it for, like, half a year.
3. Open the Battery and Brightness applet.

OBSERVED RESULT
See the screenshot.

EXPECTED RESULT
Should be somehow more readable, and not redundant.

SOFTWARE/OS VERSIONS
Operating System: Arch Linux
KDE Plasma Version: 5.23.80
KDE Frameworks Version: 5.87.0
Qt Version: 5.15.2
Kernel Version: 5.14.6-arch1-1 (64-bit)
Graphics Platform: X11
Processors: 8 × Intel® Core™ i7-6700HQ CPU @ 2.60GHz
Memory: 15.6 GiB of RAM
Graphics Processor: NVIDIA GeForce GTX 970M/PCIe/SSE2

ADDITIONAL INFORMATION
ASUS ROG has a very shitty battery controller, not kidding.
Comment 1 Nate Graham 2021-09-24 18:45:13 UTC
One issue per bug report please. :)

Let's use this bug report for the issue of the duplicated info. Can you file another one for the sub-par label formatting?
Comment 2 Nate Graham 2022-01-04 17:55:17 UTC
Git commit bafd2686c25569ac9ab52b01751214254b9e489a by Nate Graham.
Committed on 04/01/2022 at 17:53.
Pushed by ngraham into branch 'master'.

applets/batterymonitor: don't show both battery health and broken message

They both say the same thing, but the "broken battery" message is more
descriptive and actionable, so we can show only that.
FIXED-IN: 5.24

M  +1    -0    applets/batterymonitor/package/contents/ui/BatteryItem.qml

https://invent.kde.org/plasma/plasma-workspace/commit/bafd2686c25569ac9ab52b01751214254b9e489a