cblack added inline comments.

INLINE COMMENTS

> broulik wrote in Action.qml:156
> Why are you going through the prototype? Or is `children` not a "proper" 
> Array?
> Also, can we use spread operator `...` here?
> And yes, please make those a bit more readable by using useful line breaks 
> and using parentheses and braces.

It's a QML `list`, not an ECMAScript `Array`. Also, I don't see why you would 
need to use the spread operator here.

REPOSITORY
  R169 Kirigami

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

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

Reply via email to