D27996: Fix undefined check in global drawer menu mode

[Nicolas Fella]

nicolasfella marked an inline comment as done.
nicolasfella added inline comments.

INLINE COMMENTS

> apol wrote in PageRowGlobalToolBarUI.qml:49
> Wouldn't it be easier to do:
> 
>   visible: !Kirigami.Settings.isMobile && applicationWindow().globalDrawer && applicationWindow().globalDrawer.isMenu
> 
> ?
> 
> Otherwise we better check for the property with `"isMenu" in applicationWindow().globalDrawer`

The former gives `Unable to assign [undefined] to bool`

REPOSITORY
  R169 Kirigami

BRANCH
  fixkamoso

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

To: nicolasfella, #kirigami, mart, ngraham
Cc: apol, ngraham, plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, ahiemstra, davidedmundson, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200311/b4bf674d/attachment.html>