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

luebking added a comment.


  I'll try to reason that out, because I think there's some confusion about the development of the topic.
  
  After the feature was removed but before the actual release there was a decision to feature freeze KDE 4.x which tied everybodies hands and sparked the "intense" debate. Afaik this decision was not influenced by Martin and it was certainly unknown to me - I would never have signed off on the patch (and some others) if I had known that this would be a terminal change - even w/o the prospect that it would take quite some time before KDE 5 was released (and by this I don't mean "usable"). It's been 6 years and I'm *still* quite pissed about this unannounced last minute global freeze decision.
  
  Martins decision to "freeze" the PW (and iirc DG) code is unrelated from that and based on the *fact* that the code (back then, I've not looked at it since 2014/15 - everything that follows assumes that the PW code did not fundamentally change) was a convoluted mess. This isn't tied to an entry in a combobox, but the mood was that instead of tampering w/ the existing code in a hopeless effort to fix it, this desperately needed a restart. If the code hasn't been rewritten in the meantime, this is still true today (not sure whether I still have them, but I lost some hours in patches, trying to deal w/ the multiple problems, to this conclusion - yet I agree, in part because I lost some hours trying to deal w/ the multiple problems …)
  
  Re-adding the feature won't make things much worse, but you're in a way giving up and simply arrange w/ a barely maintainable source of problems which you will encounter if you try to adjust the behavior to deal w/ the suggested UX questions.
  Simply re-adding it to deal with the situation might therefore be the popular, but not the right choice. And that's not the same.

REPOSITORY
  R108 KWin

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

To: ngraham, #kwin, davidedmundson, broulik
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