D21083: [effects/presentwindows] Allow closing windows on middle-click

ngraham added a comment.


  In D21083#465601 <https://phabricator.kde.org/D21083#465601>, @zzag wrote:
  
  > We still maintain the present windows effect, we still fix bugs, but not extend it.
  
  
  //Why not?//
  
  So far I have heard a lot of vague reasons ("code is fragile" "we want to rewrite it") but no //specific// reasons. I understand those general concerns but I don't think they should be de facto blockers. It's very frustrating to be blocked by generalities and not have this patch evaluated on its merits. This is a trivial patch that does nothing more than add back code which was previously working just fine.
  
  > On the other note, I don't like that the new option is off by default. It will definitely have discoverability problems!
  
  That's because it's basically a hidden feature for experts. Those are the people who want this.

REPOSITORY
  R108 KWin

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

To: ngraham, #kwin, davidedmundson, broulik, zzag
Cc: abetts, apol, zzag, luebking, kossebau, graesslin, kwin, jraleigh, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, alexeymin, lesliezhai, ali-mohamed, hardening, jensreuterberg, sebas, mart