D22830: [kcmkwin/kwineffects] Rework the Effects KCM

filipf accepted this revision.
filipf added a comment.


  Nice work.
  
  There's a binding loop that doesn't look trivial to solve, but doesn't seem to affect anything.
  
  `file:///usr/share/kpackage/kcms/kcm_kwin_effects/contents/ui/main.qml:96:19: QML SwipeListItem: Binding loop detected for property "implicitHeight"`

REPOSITORY
  R108 KWin

BRANCH
  arcpatch-D22830

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

To: davidre, #kwin, #vdg, filipf, GB_2
Cc: ngraham, GB_2, filipf, kwin, LeGast00n, fmonteiro, sbergeron, jraleigh, fbampaloukas, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, mart