D16715: make global menu minimize state aware

[Anthony Fieroni]

anthonyfieroni added inline comments.

INLINE COMMENTS

> appmenumodel.cpp:245-248
> +        if (info.isMinimized()) {
> +            setMenuIsHidden(true);
> +        } else {
> +            setMenuIsHidden(false);

setMenuHidden(info.isMinimized());

> appmenumodel.h:42
>      Q_PROPERTY(bool menuAvailable READ menuAvailable WRITE setMenuAvailable NOTIFY menuAvailableChanged)
> +    Q_PROPERTY(bool menuIsHidden READ menuIsHidden NOTIFY menuIsHiddenChanged)
>  

Naming should be menuHidden, setMenuHidden if we follow rules.

REPOSITORY
  R120 Plasma Workspace

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

To: mvourlakos, #plasma, broulik
Cc: anthonyfieroni, 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/plasma-devel/attachments/20181106/1ea8ef03/attachment.html>