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
