Bug 423761 - Make Power Management KCM open in System Settings
Summary: Make Power Management KCM open in System Settings
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Battery Monitor (show other bugs)
Version: 5.19.2
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Kai Uwe Broulik
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-07-01 18:59 UTC by popov895
Modified: 2021-04-23 17:05 UTC (History)
4 users (show)

See Also:
Latest Commit: https://invent.kde.org/plasma/plasma-workspace/commit/ff647d4efe10a8df863199bf0b39b3e064cd57ee
Version Fixed In: 5.22

Attachments
Power Management settings (224.80 KB, video/mp4)
2020-07-01 18:59 UTC, popov895 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description popov895 2020-07-01 18:59:29 UTC 
Created attachment 129827 [details]
Power Management settings

SUMMARY

Make Power Management KCM open in System Settings

STEPS TO REPRODUCE

1. Open "Battery and Brightness" applet
2. In the top-right corner click on the "Configure Power Saving..." button

OBSERVED RESULT

Power Management KCM opens in separate window

EXPECTED RESULT

Power Management KCM opens in System Settings

SOFTWARE/OS VERSIONS

Linux/KDE Plasma: KDE neon 5.19 User Edition
(available in About System)
KDE Plasma Version: 5.19.2
KDE Frameworks Version: 5.71.0
Qt Version: 5.14.2
Comment 1 popov895 2020-07-01 19:30:12 UTC 
Same for actions in the context menu
Comment 2 Bug Janitor Service 2020-07-01 21:14:47 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/133
Comment 3 Nate Graham 2020-07-01 22:50:37 UTC 
Technically this isn't one KCM that's opened, but rather three. YOu can see because the window's sidebar has three items in it. We can only open single KCMs in System Settings, not multiple KCMs. So to do this properly, we will need to collapse those three KCMs into one first. This is desperately needed anyway, as those KCMs are quite old-fashioned and have poor UX.
Comment 4 Justin Zobel 2020-11-13 03:07:37 UTC 
Merge request closed without merging, removing Assigned status.
Comment 5 Nate Graham 2020-11-13 03:47:08 UTC 
This isn't doable until the three QWidgets-based power KCMs are collapsed into one new QML-based one. ...At which point doing what this bug report asks for is trivial. :)
Comment 6 Nate Graham 2021-04-23 17:03:21 UTC 
Actually what was I thinking? Opening one KCM in system Settings is fine as long as the others are visible in the sidebar--as they are, same as in the current multi-KCM UI. I should have merged your MR. :( Will do so manually in a moment...
Comment 7 Nate Graham 2021-04-23 17:05:11 UTC 
Git commit ff647d4efe10a8df863199bf0b39b3e064cd57ee by Nate Graham, on behalf of Eugene Popov.
Committed on 23/04/2021 at 17:04.
Pushed by ngraham into branch 'master'.

[applets/batterymonitor] Open Powerdevil KCMs in System Settings

This is generally what we do for other KCMs, and the UX is virtually
unchanged from the current multi-KCM dialog, with the exception of being
able to navigate elsewhere if you want.
FIXED-IN: 5.22

M  +3    -6    applets/batterymonitor/package/contents/ui/batterymonitor.qml

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