cfeck added inline comments. INLINE COMMENTS
> ktoggleshowmenubaraction.cpp:21 > + > +#include "ktoggleshowmenubaraction.h" > + Do we still follow the rule "include own headers first"? > ktoggleshowmenubaraction.h:28 > + * An action to hide or show the menubar of a window. > + * The action by default remind the user how to show back > + * the menu bar using the keyboard shortcut. reminds Additionally, "show back" does not sound like proper english, but I am not a native english speaker. REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D17013 To: lsartorelli, ngraham Cc: cfeck, broulik, kde-frameworks-devel, michaelh, ngraham, bruns