D16836: [effects] Split the Fade effect

ngraham added a comment.


  In D16836#358785 <https://phabricator.kde.org/D16836#358785>, @graesslin wrote:
  
  > In D16836#358375 <https://phabricator.kde.org/D16836#358375>, @ngraham wrote:
  >
  > > Very cool, +1 for putting these effects into an exclusive group.
  > >
  > > Have we given any thought to how to better present the exclusive groups in the KCM? Right now it's not as clear as it could be.
  >
  >
  > As I spent quite some time on that: it's difficult to present this in a good way and I implemented the best solution I found. If we want to present it better we might have to completely rethink the effects kcm, but we also did that several times. The big problem is that it's exclusive group which could also contain none. That's from a UI perspective really difficult.
  
  
  Adding a simple "None" entry for exclusive groups that need one seems like it could work. Also, something as simple as more whitespace before the exclusive group's header and after the last entry might help. Right now the exclusive groups kind of blend into the other non-exclusive entries.

REPOSITORY
  R108 KWin

BRANCH
  split-fade-effect

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

To: zzag, #kwin, #plasma, #vdg, graesslin
Cc: graesslin, abetts, ngraham, kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, sebas, apol, mart