D16232: Always keep group transients above

davidedmundson added a comment.


  I generally hold the view that we shouldn't work round KDE clients, but I'm very unconvinced in this case.
  
  As for what kmail should have set. It's not modal, which means the author explicitly wanted the user to be able to interact with the main window whilst it is open. 
  Kmail does not set a transient, QtXCB does because it's of the window type.
  
  In the bug reports listed, if you follow the dupes there are 3 applications affected.
  
  -------
  
  Your quote is not from the spec, but in a compatibility notes section under the heading " the following layered stacking order is recommended, ".  
  The line after is  "The window manager may choose to put some windows in different stacking positions"
  
  -------
  
  Working with an old code base can suck, but the huge advantage is that we have the real-world experience with quirks and corner cases.
  
  Out of curiosity, does this search window still appear in the task manager? That was cited as one of the rationals for the change.

REPOSITORY
  R108 KWin

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

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