----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102261/ -----------------------------------------------------------
Review request for kwin, Plasma, Aaron J. Seigo, Martin Gräßlin, and Farhad Hedayati Fard. Summary ------- In the farhad_hf/lockscreen branch (that moves screen locking functionality from krunner to kwin), the shortcut for locking the screen (Ctrl+Alt+L by default) remains in KRunner, which will not necessarily be running (when plasma is in "netbook" mode, for example). This moves the shortcut registration to kwin (including a necessary hack for unregistering krunner's claim on it). (See also https://git.reviewboard.kde.org/r/101943/ ). I can also merge latest master into the branch and/or add in a KWIN_BUILD_SCREENSAVER cmake option (as in the original review request) if you (Martin/Farhad) want. Diffs ----- krunner/krunnerapp.cpp 55ac5ca kwin/screenlocker/screensaver/saverengine.h 3384d4a kwin/screenlocker/screensaver/saverengine.cpp 6c15be6 kwin/useractions.cpp 44685b9 Diff: http://git.reviewboard.kde.org/r/102261/diff Testing ------- Thanks, Alex
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel