----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102261/ -----------------------------------------------------------
(Updated Aug. 9, 2011, 2:42 p.m.) Review request for kwin, Plasma, Aaron J. Seigo, Martin Gräßlin, and Farhad Hedayati Fard. Changes ------- Remove the existing shortcut registration. 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 (updated) ----- krunner/krunnerapp.cpp 55ac5ca kwin/kwinbindings.cpp 40bba26 kwin/screenlocker/screensaver/saverengine.h 3384d4a kwin/screenlocker/screensaver/saverengine.cpp 6c15be6 kwin/useractions.cpp 44685b9 kwin/workspace.h 2b4c8e9 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
