D15644: Provide option to hide menu bar for Ksysguard

Harald Sitter noreply at phabricator.kde.org
Wed Sep 26 14:42:42 BST 2018


sitter added a comment.


  Looks almost perfect to me. Only nit-pick I have is the fact that the include is out of order. Other than that it looks awesome 👍

INLINE COMMENTS

> ksysguard.cpp:54
>  #include <QStatusBar>
> +#include <QMenuBar>
>  

That should be sorted alphabetically.

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

To: lsartorelli, ngraham, #plasma, #frameworks
Cc: sitter, acrouthamel, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180926/156c3c19/attachment.html>


More information about the Kde-frameworks-devel mailing list