> 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
> 
>

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to