broulik added inline comments.

INLINE COMMENTS

> ActionToolBarLayoutDetails.qml:110
> +
> +    onWidthChanged: { delayUpdate.start = true }
> +    Component.onCompleted: { delayUpdate.start = true }

How can this even work? `start` is a method. You probably meant `running = 
true`? or `start()` which, in contrast to a `QTimer` will *not* restart the 
timer if already running.

So apparently this isn't necessary at all, if it not doing anything actually 
works?

REPOSITORY
  R169 Kirigami

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

To: apol, #kirigami
Cc: broulik, plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, ngraham, 
apol, ahiemstra, davidedmundson, mart, hein

Reply via email to