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