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

Reply via email to