D16228: Raise window group together with group transient

graesslin added a comment.


  In D16228#353941 <https://phabricator.kde.org/D16228#353941>, @zzag wrote:
  
  > In D16228#353940 <https://phabricator.kde.org/D16228#353940>, @graesslin wrote:
  >
  > > I would say: don't fix what is not broken.
  >
  >
  > Well, no, it's broken. Why do we allow to have the following sandwich with a group transient but not with a transient for a toplevel window?
  
  
  It's a bug in the effects kcm. It should be a transient to systemsettings. It was not the intention of the authors to make them group transients, but transients. Don't workaround bugs, fix the real ones :-)
  
  That's my point: like in the effects kcm case I doubt the group transients are the intention of the authors. It's rather an unwanted side effect of API missusage.

REPOSITORY
  R108 KWin

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

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