Review Request 125678: Introduce User Switcher plasmoid

[Kai Uwe Broulik]

> On Okt. 18, 2015, 9:26 vorm., Heiko Tietze wrote:
> > Nice user switcher. Question is whether or not the user icon is to show in the panel (clutters IMHO, so done right) and what happens when you click one of the global functions. The Lock/Logout plasmoid shows a confirmation bar with reboot, shut down, and log out (the latter enabled by default). Is this bar shown for leave? And actually I'd rather keep plasmoids simple and remove all features that are not directly related to user sessions. So it should be taken into consideration to remove Lock and Leave... (maybe add just logout).
> > 
> > And last but not least you could think about the sort order. If there are really more than one instance of the same user as in your screenshot, the current user may want to switch between own instances.
> > 
> > PS: New users may expect such a dialog for activities. Does mixing sessions and activities make sense?
> 
> Kai Uwe Broulik wrote:
>     > Question is whether or not the user icon is to show in the panel
>     
>     I was thinking about it but decided against it because, yes, it clutters it.
>     > Is this bar shown for leave?
>     
>     Yes, pressing "Leave..." will open the Shutdown/Reboot/Logout dialog, that part is actually taken from the lock logout applet :)
>     > not directly related to user sessions
>     
>     Locking the screen and logging off is related to the *current* session.
>     > remove Lock and Leave...
>     
>     I don't think telling someone "if you want to switch to the other user, click here, but if you want to just log out or lock the screen, click here, here, and here" (kickoff menu eg)
>     > If there are really more than one instance of the same user as in your screenshot, the current user may want to switch between own instances
>     
>     Makes sense, ie. sort the current user's sessions first, then the rest alphabetically? Or by VT (F1, F2, ..)? However, the sort order is not determined by the plasmoid but the separate model (Review 124980) and would (eventually) affect the regular user switcher and the one on the lock screen.
> 
> Heiko Tietze wrote:
>     | Yes, pressing "Leave..." will open the Shutdown/Reboot/Logout dialog, that part is actually taken from the lock logout applet :)
>     
>     OT: This confirmation bar shows "logout" (not "leave") by default, even when I clicked shutdown in the refering plasmoid.
>     
>     Waiting for other replies...

You can configure what the bar should default to in System Settings -> Startup & Shutdown -> Desktop Session -> Default setting for logging off. It defaults to "Log out" (I think it should be Shutdown), or depending on whether other users are currently logged in.


- Kai Uwe


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


On Okt. 17, 2015, 6:11 nachm., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125678/
> -----------------------------------------------------------
> 
> (Updated Okt. 17, 2015, 6:11 nachm.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Repository: kdeplasma-addons
> 
> 
> Description
> -------
> 
> It is prety common in other desktop environments to have something that allows switching between users and logout/shutdown quickly. It is inspired by the Fast User Switch plasmoid from kde-apps.org [1]
> 
> [1] http://kde-apps.org/content/show.php?content=127462
> 
> 
> Diffs
> -----
> 
>   applets/userswitcher/package/metadata.desktop PRE-CREATION 
>   applets/userswitcher/package/contents/ui/main.qml PRE-CREATION 
>   applets/userswitcher/package/contents/ui/ListDelegate.qml PRE-CREATION 
>   applets/userswitcher/Messages.sh PRE-CREATION 
>   applets/CMakeLists.txt 4079726 
>   applets/userswitcher/CMakeLists.txt PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/125678/diff/
> 
> 
> Testing
> -------
> 
> When the applet opens, it updates. It grows a maximum of half the screen height, should there be more users currently logged in, the list can be scrolled. Switching users works, both with and without locking the screen, depending on user's settings. Opening new sessions, locking the screen and shutting down also work.
> 
> The applet doesn't have any settings, the label probably needs tweaking so its sizing policy is similar to the digital clock so the two look reasonable next to each other.
> 
> 
> File Attachments
> ----------------
> 
> Plasmoid in action
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/10/17/05d93363-9b15-4c77-abe0-6f8ed9b3cab7__userswitcherplasmoid3.png
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151018/16e0d3a5/attachment-0001.html>