Bug 360276 - Battery Plasmoid review
Summary: Battery Plasmoid review
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Power and Battery widget (other bugs)
Version First Reported In: 5.5.3
Platform: Other Other
: NOR normal
Target Milestone: 1.0
Assignee: Kai Uwe Broulik
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-03-08 23:50 UTC by andreas
Modified: 2018-10-31 03:05 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments
battery plasmoid (999.17 KB, image/png)
2016-03-08 23:50 UTC, andreas
Details

Note You need to log in before you can comment on or make changes to this bug.
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