> On April 20, 2014, 9:18 p.m., Christoph Feck wrote: > > Can someone more familiar with the screen locker please review this, or > > suggest someone who could review it? > > Christoph Feck wrote: > Felix, could you please add Wolfgang Bauer (wbauer) and maybe Thomas > Lübking (luebking) to the reviewers?
I can at least confirm that this patch fixes the issue for me as well. - Wolfgang ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117499/#review56088 ----------------------------------------------------------- On April 22, 2014, 12:53 p.m., Felix Geyer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117499/ > ----------------------------------------------------------- > > (Updated April 22, 2014, 12:53 p.m.) > > > Review request for Plasma, Thomas Lübking and Wolfgang Bauer. > > > Bugs: 331761 > http://bugs.kde.org/show_bug.cgi?id=331761 > > > Repository: kde-workspace > > > Description > ------- > > The user switichting dialog from the screen locker is destroyed when the > cancel button is pressed. > As a result you can't open it again. > > The screen locker uses a PlasmaComponents.PageStack to show the password > input and user switching dialogs. > When a dialog is popped from the stack it is destroyed by PageStack. > > This change fixes this by creating a new instance of userSessionsUI before it > is pushed to the stack. > Additionally the switchUserEnabled property is set once when the screen > locker is created as userSessionsUIInstance > doesn't always exist. > > > Diffs > ----- > > > ksmserver/screenlocker/greeter/themes/org.kde.passworddialog/contents/ui/main.qml > 0d3c7d5 > > Diff: https://git.reviewboard.kde.org/r/117499/diff/ > > > Testing > ------- > > - Lock the screen > - Press "Switch user" > - Press "Cancel" > - With this patch the "Switch user" button is visible and functional. > > > Thanks, > > Felix Geyer > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
