----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119814/ -----------------------------------------------------------
(Updated Jan. 5, 2015, 10:46 a.m.) Review request for Plasma and Àlex Fiestas. Changes ------- * added fallback through solid DBus signal. Note: this is untested as my system has logind. * rebased to current master Repository: plasma-workspace Description ------- [ksld] ScreenLocker inhibits sleep on logind When the system is going to sleep we want to ensure that the screen gets locked before the system goes to sleep. Logind provides the inhibitor locks which can be used for this. If logind is available ksld gains an inhibitor lock for sleep when the screen is unlocked. As soon as the screen gets locked the inhibitor lock is released. In addition it connects to the prepareForSleep signal by logind and locks the screen. The solution needs to be extended to have a config option whether the screen should be locked on sleep. Currently this is provided by powerdevil. Also the solution can only work properly if power devil uses logind's sleep dbus interface. [ksld] Don't block till the greeter is started We only want to ensure that the greeter gets started. There is no need to block for that. Instead we can connect to the error signal and unlock in case the greeter failed to start. ----- @Alex: what do you think is the best solution for handling the "lock screen on resume" config option? My idea would be to move it to screen locker and expose the value through DBus, so that powerdevil can still read it. Diffs (updated) ----- ksmserver/screenlocker/CMakeLists.txt 83a5b39e3373ffef5e03218d2047ab0250b7c448 ksmserver/screenlocker/autotests/CMakeLists.txt 4bff1c6b1d8fc360197c422f8d036dff3eae5efe ksmserver/screenlocker/kcfg/kscreenlockersettings.kcfg a0253d687150702aa4c22200d9f6a577d0cab6be ksmserver/screenlocker/kcm/kcm.ui 8f8654b5fe34b8e4bfd95f652659a59c6c664a55 ksmserver/screenlocker/ksldapp.h a606aa84cb5d9628d6e8a68360b53ddc483c2778 ksmserver/screenlocker/ksldapp.cpp addf49e2f9cada9f1c0c69bc78837a30a7ab77e0 ksmserver/screenlocker/logind.h a335ddc2f6f55b071f824f9da94652a4dd70c483 ksmserver/screenlocker/logind.cpp dcfc7f321b3cf29ef68aac8006aa37f5e4e00956 Diff: https://git.reviewboard.kde.org/r/119814/diff/ Testing ------- Thanks, Martin Gräßlin
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel