D16293: [wayland] Set better placement for XDG Toplevels

davidedmundson created this revision.
davidedmundson added a reviewer: KWin.
Herald added a project: KWin.
Herald added a subscriber: kwin.
davidedmundson requested review of this revision.

REVISION SUMMARY
  XDG Popups will have a transient parent and positional information.
  XDG Toplevels can have a transient parent without having positional
  information.
  
  Currently we set that we have a transient placement hint of 0,0 which
  means the newly opened children go to the top left of the parent.
  
  This new code paths treat child top levels as dialogs centering them to
  the parent.
  
  BUG: 383167

TEST PLAN
  Dolphin help->about
  Appeared where it does on X

REPOSITORY
  R108 KWin

BRANCH
  master

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

AFFECTED FILES
  placement.cpp
  shell_client.cpp

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