davidedmundson added inline comments.

INLINE COMMENTS

> ngraham wrote in ExpandableListItem.qml:2
> Where should it live? PC2? Kirigami?

Definitely can't be in PC2.

Kirigami is doable, but it'd require some code changes to not use any Plasma.* 
imports and go through relevant abstractions

Failing that plasmaextracomponents

> ngraham wrote in ExpandableListItem.qml:23
> Not unused, it's needed for `Highlight`, which has no PC3 version.

I meant this one

import org.kde.plasma.plasmoid 2.0

having that in library code is surprising.

> ngraham wrote in ExpandableListItem.qml:230
> Not random; this is the easing curve we agreed to and is now in the HIG: 
> https://hig.kde.org/style/animations.html#guidelines

I meant the duration

REPOSITORY
  R242 Plasma Framework (Library)

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

To: ngraham, #vdg, #plasma, davidedmundson
Cc: davidedmundson, bruns, niccolove, cblack, davidre, kde-frameworks-devel, 
LeGast00n, GB_2, michaelh, ngraham

Reply via email to