----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128854/#review98998 -----------------------------------------------------------
Just as I'd imagined it, +1 from the usability side! - Thomas Pfeiffer On Sept. 7, 2016, 12:24 p.m., Gregor Mi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128854/ > ----------------------------------------------------------- > > (Updated Sept. 7, 2016, 12:24 p.m.) > > > Review request for KDE Base Apps, Martin Gräßlin, John Tapsell, Ken Vermette, > Thomas Lübking, and Thomas Pfeiffer. > > > Repository: libksysguard > > > Description > ------- > > This adds a new "Tools" button to the ksysguard widget which contains entries > to related tools: > > - Run Command --> starts KRunner to execute a command > - KSysguard --> starts KSysguard (this is useful because Ctrl+Esc only starts > the lightweight System Monitor which has less features) > - Info Center --> starts the Info Center which shows additional system > information > - Htop --> starts htop > - Kill a window (Ctrl+Alt+Esc) --> triggers the KWin kill window function. > The displayed shortcut is the one set in Global Keyboard Shortcuts. > Currently, this shortcut is shown in the End Process... button tooltip but > there it is hard-coded. > > This RR replaces this old RR https://git.reviewboard.kde.org/r/122249/ > > > Diffs > ----- > > CMakeLists.txt c7145d405c5264bf356469d7c64c95aa3d327627 > processui/CMakeLists.txt 324178a5abe318b6bb0cdcde19f5788ae356bcb9 > processui/ProcessWidgetUI.ui e50f55cf1813b00d49b1716023df487ffbd536e3 > processui/ksysguardprocesslist.cpp 44603de029323f9605cd0db91da32aa23e8ed98f > > Diff: https://git.reviewboard.kde.org/r/128854/diff/ > > > Testing > ------- > > - test if all tools start correctly > - check if a changed shortcut for Kill Window is shown in the tools menu (see > screenshot) > - check if for example htop is not installed that there is no menu item for it > > > File Attachments > ---------------- > > screenshot of the new Tools button > > https://git.reviewboard.kde.org/media/uploaded/files/2016/09/07/ac70c386-6665-4898-94e7-b256759f5a5e__screenshot_20160907_140935.png > > > Thanks, > > Gregor Mi > >