ratijas added inline comments.

INLINE COMMENTS

> ExpandableListItem.qml:269
> +     */
> +    property bool isEnabled: true
> +

Why simply Item::enabled wasn't enough? Is there a real use-case that justifies 
an extra property with overlapping name? Should be documented if there is one.

REPOSITORY
  R242 Plasma Framework (Library)

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

To: ngraham, #vdg, #plasma, davidedmundson
Cc: ratijas, meven, dfaure, mart, davidedmundson, bruns, niccolove, cblack, 
davidre, kde-frameworks-devel, LeGast00n, michaelh, ahmadsamir, ngraham, vkrause

Reply via email to