D10297: Add new "Tools" button above System Monitor's process list

gregormi noreply at phabricator.kde.org
Wed Mar 7 10:42:08 UTC 2018


gregormi added a comment.


  In D10297#207127 <https://phabricator.kde.org/D10297#207127>, @rkflx wrote:
  
  > Just tried the patch. I think for System Activity / [Ctrl] + [⎋] the button is fine (annoyingly with a slightly larger height than the combobox, but that's life…). For KSysGuard itself it looks a bit odd, why would it appear exactly on this tab page? I could imagine the menubar would be a better place.
  >
  > (Sorry this comes so late, I only reviewed in more detail because nobody from Plasma bothered so far. Ignore my comment if you want.)
  
  
  Some people from Plasma reviewed the original review request on reviewboard. Should we add some more reviewers before it can go in?

REPOSITORY
  R111 KSysguard Library

REVISION DETAIL
  https://phabricator.kde.org/D10297

To: gregormi, #plasma
Cc: anthonyfieroni, andreaska, rkflx, ngraham, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180307/7451c8ce/attachment.html>


More information about the Plasma-devel mailing list