----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/109609/#review46506 -----------------------------------------------------------
Ship it! Ship It! - Lukáš Tinkl On Sept. 11, 2013, 1:15 a.m., Wolfgang Bauer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/109609/ > ----------------------------------------------------------- > > (Updated Sept. 11, 2013, 1:15 a.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 > >
