Review Request 124585: Add pretty user switcher

andreas kainz kainz.a at gmail.com
Thu Nov 12 11:14:51 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124585/#review88285
-----------------------------------------------------------


Hi Kai Uwe, I start an forum thread about the design. this isn't a showstopper for that request. I only want to let you know and hope you can join the discussion https://forum.kde.org/viewtopic.php?f=285&t=129283

- andreas kainz


On Nov. 8, 2015, 6:14 nachm., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124585/
> -----------------------------------------------------------
> 
> (Updated Nov. 8, 2015, 6:14 nachm.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> This adds a user switcher dialog borrowing elements from the lock screen and log out dialog. I never understood why KRunner was used for that.
> 
> It is not yet hooked up to the system (would make KRunner's "switchUser" interface forward that to KSMServer so everybody using it (eg. kickoff) would continue to work without adjustments). To test it run qdbus org.kde.ksmserver /KSMServer openSwitchUserDialog
> 
> When it opens it defaults to "New Session", from an interaction POV the list view behaves like the one on the lock screen.
> 
> It locks the screen when switching users only when the user has enabled the lock screen (might need a separate option for that?) That unconditional locking on session switching has turned out to be a major annoyance.
> 
> 
> Diffs
> -----
> 
>   ksmserver/CMakeLists.txt b4eb9b8 
>   ksmserver/org.kde.KSMServerInterface.xml 3680ed7 
>   ksmserver/server.h 87e46cb 
>   ksmserver/server.cpp ffb15d2 
>   ksmserver/switchuserdialog.h PRE-CREATION 
>   ksmserver/switchuserdialog.cpp PRE-CREATION 
>   lookandfeel/contents/components/UserList.qml 778d8ea 
>   lookandfeel/contents/components/UserSelect.qml 5af566d 
>   lookandfeel/contents/userswitcher/UserSwitcher.qml e69de29 
> 
> Diff: https://git.reviewboard.kde.org/r/124585/diff/
> 
> 
> Testing
> -------
> 
> Looks pretty, switching to a new or existing session works.
> 
> Btw, weren't we able to click outside the dialog to dismiss it in 4.x? Seems broken for the logout dialog as well.
> 
> 
> File Attachments
> ----------------
> 
> No active sessions
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/11/08/6acd1764-db79-401b-936b-7a6fb5809c5a__userswitchera1.jpg
> Switch sessions
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/11/08/b21de001-c011-4733-a47f-e2966ac2b955__userswitchera2.jpg
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151112/1d2ea661/attachment.html>


More information about the Plasma-devel mailing list