D13084: Avoid invalid geometry of internal clients through plasma surface interface

romangg added a comment.


  In D13084#319377 <https://phabricator.kde.org/D13084#319377>, @davidedmundson wrote:
  
  > I have explained multiple times above.
  
  
  But you haven't. You did "some investigation" without coming up with a proper overall solution, instead throwing three solution ideas at the wall, maybe none of them working in the end, but certainly not doable until 5.14 beta. You can prove me otherwise if you want to. Would be happy about it.
  
  > You can't just ignore my comments and badger me repeatedly till I give in and accept it anyway.
  
  If it felt this way to you, tell me. But from my side I gave you over a month to come up with a different solution, so I'm not overly badgering you.
  
  > The initial investigation was wrong, the rationale was wrong, the patch is wrong, you did *nothing* to address any of this except repeatedly push me to accept this anyway??!
  
  This patch is there to solve a specific bug people complained about. In my tests it works without problems, so let's roll with it until we have a "right investigation" with a "right rationale" and a "right patch", whatever this means.
  
  > I do not remotely owe you an apology.
  
  Think what you want.

REPOSITORY
  R108 KWin

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

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