----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/109609/#review46637 -----------------------------------------------------------
This review has been submitted with commit c014af7836a9fa7df8a17485f9337c35e6d535de by Wolfgang Bauer to branch KDE/4.11. - Commit Hook On Sept. 10, 2013, 11:15 p.m., Wolfgang Bauer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/109609/ > ----------------------------------------------------------- > > (Updated Sept. 10, 2013, 11:15 p.m.) > > > Review request for kde-workspace and Oliver Henshaw. > > > Bugs: 314859 > http://bugs.kde.org/show_bug.cgi?id=314859 > > > Repository: kde-workspace > > > Description > ------- > > This patch disables session management for screensavers. > Therefore no screensaver windows popup anymore on login if processes keep on > running when the user logs out. > > As proposed in https://bugs.kde.org/show_bug.cgi?id=314859#c18 > > This doesn't solve the underlying problem of bug#224200 of course, but > provides better user experience for now, so please accept. > > And IMHO it doesn't make sense to restore screensavers on login, anyway. > > > Diffs > ----- > > kscreensaver/libkscreensaver/main.cpp 6851ba1 > > Diff: https://git.reviewboard.kde.org/r/109609/diff/ > > > Testing > ------- > > - Turn on the "Blank Screen" (kblankscrn.kss) screensaver in systemsettings. > - Wait for the screensaver to kick in > - Verify that the process is still running (due to bug#224200) > - log out > - log in again > Now no "kblankscrn.kss" window is opened anymore (without the patch, a window > should appear) > > > Thanks, > > Wolfgang Bauer > >
