Bug 360276

Summary:	Battery Plasmoid review
Product:	[Plasma] plasmashell	Reporter:	andreas <kainz.a>
Component:	Power and Battery widget	Assignee:	Kai Uwe Broulik <kde>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bugseforuns, kde, mklapetek, nate, plasma-bugs-null
Priority:	NOR
Version First Reported In:	5.5.3
Target Milestone:	1.0
Platform:	Other
OS:	Other
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	battery plasmoid

Description andreas 2016-03-08 23:50:37 UTC

the sizing is prety good maybe a link to the kinforcenter (or an kcm module) to the battery consumtion stuff would be awesome.

don't show the user in the widget something like that:

"No screen or keeboard brightness contrast available"

maybe you can add this to the kcm where you give the user the feedback that something is not available but not in the main plasmoid (we don't show the user SOMETHING IS BROKEN at least not when it is only not available.

thange the widget icon to the "battery" icon please which is available in breeze.

Reproducible: Always

Comment 1 andreas 2016-03-08 23:50:57 UTC

Created attachment 97778 [details]
battery plasmoid

Comment 2 Kai Uwe Broulik 2016-03-09 09:52:24 UTC

Git commit 412b6f90402c0d2ccbcf5d8879a582ffaf67e171 by Kai Uwe Broulik.
Committed on 09/03/2016 at 09:51.
Pushed by broulik into branch 'Plasma/5.6'.

[Battery Monitor] Drop "You don't have any brightness controls" text

M  +0    -13   applets/batterymonitor/package/contents/ui/PopupDialog.qml

http://commits.kde.org/plasma-workspace/412b6f90402c0d2ccbcf5d8879a582ffaf67e171

Comment 3 Kai Uwe Broulik 2016-03-09 09:53:36 UTC

Breeze is missing the appropriate battery status icons Oxygen has.

Comment 4 Kai Uwe Broulik 2016-03-09 11:40:25 UTC

Git commit abb3183c27cea308676bdb6a56b552a3a3a06dfd by Kai Uwe Broulik.
Committed on 09/03/2016 at 11:39.
Pushed by broulik into branch 'Plasma/5.6'.

[Battery Monitor] Just use "battery tooltip icon

There's no point in having separate logic for the tooltip and Breeze
doesn't have the icons we need anyway.

M  +0    -5    applets/batterymonitor/package/contents/code/logic.js
M  +1    -2    applets/batterymonitor/package/contents/ui/batterymonitor.qml

http://commits.kde.org/plasma-workspace/abb3183c27cea308676bdb6a56b552a3a3a06dfd

Comment 5 andreas 2016-03-10 07:05:03 UTC

I think now it is fixed isn`t it

Comment 6 Kai Uwe Broulik 2016-03-10 12:33:44 UTC

Still missing the Energy Information thing in the settings but I don't want to add it until I figure out why it crashes.

Comment 7 Martin Klapetek 2016-03-10 14:09:40 UTC

Speaking of which, can the "enable power management" checkbox be moved to the top like the other applets do? Including the settings icon?

Comment 8 David Edmundson 2018-10-31 03:05:22 UTC

> can the "enable power management" checkbox be moved to the top like the other applets do? Including the settings icon?

It is now