D25177: ActionToolBar: Automatically change to icon-only for actions marked KeepVisible

[Arjen Hiemstra]

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

INLINE COMMENTS

> broulik wrote in ActionToolBar.qml:111
> Shouldn't this be
> 
>   modelData.visible === undefined || modelData.visible
> 
> i.e. have the action show if `visible` is not defined

You are right, fixed.

REPOSITORY
  R169 Kirigami

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

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