----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125678/#review87002 -----------------------------------------------------------
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? - Heiko Tietze 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
