Bug 448797

Summary:	Inappropriate "Low battery level" icon when only batteries come from external wireless devices
Product:	[Plasma] plasmashell	Reporter:	Viorel-Cătălin Răpițeanu <rapiteanu.catalin>
Component:	Battery Monitor	Assignee:	ratijas <me>
Status:	RESOLVED FIXED
Severity:	minor	CC:	forestix, kde, me, nate, nicolas.fella
Priority:	NOR
Version:	master
Target Milestone:	1.0
Platform:	Compiled Sources
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/plasma-workspace/commit/7a40d72fcf0110c18053fb6872471ac5e801c0da	Version Fixed In:	5.25
Sentry Crash Report:
Attachments:	The Power Management indicator Expanded Power Management applet

Description Viorel-Cătălin Răpițeanu 2022-01-19 21:43:18 UTC

SUMMARY
***
When you have a wireless mouse connected, but no other method of changing the monitor brightness, the 'low battery' icon is shown for the Power Management applet.
***


STEPS TO REPRODUCE
1. Connect a wireless mouse to a desktop PC.

OBSERVED RESULT
The Power Management applet correctly identifies the wireless mouse, but the icon displayed for this applet is the 'low battery' one. The current battery charge is not taken into any consideration.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Plasma master
KDE Plasma Version: 5.23.90 master
KDE Frameworks Version: 5.91 master
Qt Version: 5.15.2

Comment 1 Viorel-Cătălin Răpițeanu 2022-01-19 21:44:42 UTC

Created attachment 145652 [details]
The Power Management indicator

It contains a single wireless device (see the other battery-indicator-with-battery attachment)

Comment 2 Viorel-Cătălin Răpițeanu 2022-01-19 21:45:12 UTC

Created attachment 145653 [details]
Expanded Power Management applet

A single device is available.

Comment 3 Nate Graham 2022-01-31 15:56:26 UTC

Git commit 7a40d72fcf0110c18053fb6872471ac5e801c0da by Nate Graham, on behalf of Aleix Pol Gonzalez.
Committed on 31/01/2022 at 15:56.
Pushed by ngraham into branch 'master'.

applets/batterymonitor: Use the same property to define hasBatteries

Otherwise it looks weird when it gets out of sync because Has
Cumulative.
FIXED-IN: 5.24

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

https://invent.kde.org/plasma/plasma-workspace/commit/7a40d72fcf0110c18053fb6872471ac5e801c0da