D5016: Add UI for hybrid suspend mode

2017-03-11 Thread David Strobach
This revision was automatically updated to reflect the committed changes. Closed by commit R122:c6b76da7a679: Add UI for hybrid suspend mode (authored by strobach). REPOSITORY R122 Powerdevil CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5016?vs=12388&id=12390 REVISION DETAIL htt

D5016: Add UI for hybrid suspend mode

2017-03-11 Thread David Strobach
strobach added a comment. In https://phabricator.kde.org/D5016#94363, @broulik wrote: > The question is whether we should treat "hybrid suspend" and "suspend" as separate options or just have a global checkbox "[X] When possible use hybrid suspend". I don't think so. I do, for in

D5016: Add UI for hybrid suspend mode

2017-03-11 Thread Kai Uwe Broulik
broulik accepted this revision. broulik added a comment. This revision is now accepted and ready to land. Thanks a lot! Interestingly enough PowerDevil's suspend session action already had a case for hybrid suspend it just wasn't exposed in the UI. The question is whether we should t

D5016: Add UI for hybrid suspend mode

2017-03-11 Thread David Strobach
strobach added reviewers: broulik, sebas. REPOSITORY R122 Powerdevil REVISION DETAIL https://phabricator.kde.org/D5016 To: strobach, broulik, sebas Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

D5016: Add UI for hybrid suspend mode

2017-03-11 Thread David Strobach
strobach created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Add "Hybrid suspend" option to suspend actions and button handler actions. The action is already supported by Solid. REPOSITORY R122 Powe