D15187: Merge switch user dialog into lockscreen

Marco Martin noreply at phabricator.kde.org
Fri Aug 31 14:57:30 BST 2018


mart accepted this revision.
mart added a comment.
This revision is now accepted and ready to land.


  lovely code deletions :)
  and yeah, requiring the passowd again to cancel user switching kinda makes sense, it shouldn't be an issue

INLINE COMMENTS

> server.cpp:1095
> +    OrgKdeScreensaverInterface iface(QStringLiteral("org.freedesktop.ScreenSaver"), QStringLiteral("/ScreenSaver"), QDBusConnection::sessionBus());
> +    iface.SwitchUser();
>  }

should we need some confirmation from the screenlocker that the operation actually went well?

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D15187

To: davidedmundson, #plasma, mart
Cc: mart, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180831/f55e0842/attachment.html>


More information about the Plasma-devel mailing list