Bug 345940 - Move "Enable Power Management" checkbox to the top as other applets
Summary: Move "Enable Power Management" checkbox to the top as other applets
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Battery Monitor (show other bugs)
Version: master
Platform: Other Linux
: NOR wishlist
Target Milestone: 1.0
Assignee: Kai Uwe Broulik
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-07 07:10 UTC by Michi
Modified: 2017-11-02 12:58 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 Michi 2015-04-07 07:10:46 UTC
I'm actually talking about the "Enable Power Management" and PM Settings that are in the midst of a few sliders for other things.
I would recommend using the approach from the plama-nm plasmoid where similar elements are located at the top.

Reproducible: Always
Comment 1 David Edmundson 2015-04-07 09:50:53 UTC
That's not a specific bug report.
Comment 2 Martin Klapetek 2015-04-07 11:10:57 UTC
I for one actually think this is a good report.

Currently you enable/disable networks and bluetooth on the top while enabling/disabling PM is in the middle of the sliders and stuff.

I do agree that it would make more sense on the top, even if just for consistency.
Comment 3 Kai Uwe Broulik 2015-04-07 11:41:24 UTC
I do agree that the layout isn't optimal but it has only been ported to Plasma 5 verbatim before Plasma-NM had its current layout.
Comment 4 Nate Graham 2017-11-01 21:56:33 UTC
Patch available at
Comment 5 Nate Graham 2017-11-01 21:56:42 UTC
Er, at https://phabricator.kde.org/D8607
Comment 6 Nate Graham 2017-11-02 12:58:37 UTC
Git commit 727db59f21b02d9fffbbc367a6a32ea6e6282d83 by Nathaniel Graham.
Committed on 02/11/2017 at 12:58.
Pushed by ngraham into branch 'master'.

Move power management checkbox to the top

Summary:

Move the "Enable Power Management" checkbox to the top of the window to match other applets (bluetooth, network)

Reviewers: #plasma_workspaces, broulik, mck182, davidedmundson, mart

Reviewed By: #plasma_workspaces, davidedmundson, mart

Subscribers: mart, jensreuterberg, davidedmundson, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D8607

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

https://commits.kde.org/plasma-workspace/727db59f21b02d9fffbbc367a6a32ea6e6282d83