Review Request 125678: Introduce User Switcher plasmoid

[Thomas Pfeiffer]

> On Oct. 19, 2015, 8:47 a.m., Marco Martin wrote:
> > Nice! But I wonder if wouldn't be better to have the whole plasmoid as a scrolling list instead of just the users scrolling and not the rest, it looks a bit werird. I know that technically the user list and the other functions are different things, but that shouldn't matter to the user (the other buttons could be in the footer of the listview)

It might just look weird on this screenshot because the separator between the users and the leave options (which is important anyway) is covered mostly by the highlight here. This should be fixed, the separator has to be below the higlight.
It makes sense to me that the leave options are always directly visible and do not have to be scrolled to, because they are likely to be more important to a user than the 8th sessions in a list. Besides, in actual use, it's unlikely that so many sessions are running at the same time anyway (think of the memory!). Unless most of them are console sessions, and people who have lots of console sessions open at the sime time are likely to find Ctrl-Alt-FX to be more convenient for switching anyway.


- Thomas


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


On Oct. 17, 2015, 6:11 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125678/
> -----------------------------------------------------------
> 
> (Updated Oct. 17, 2015, 6:11 p.m.)
> 
> 
> 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/20151019/5252be4c/attachment-0001.html>