D18956: [wayland] Handle sizes in ShellClient::transientPlacement

zzag added inline comments.

INLINE COMMENTS

> placement.cpp:503
>      const QRect screen =  Workspace::self()->clientArea(parent->isFullScreen() ? FullScreenArea : PlacementArea, parent);
> -    const QPoint popupPos = c->transientPlacement(screen).topLeft();
> -    c->move(popupPos);
> +    const QRect popupLocation = c->transientPlacement(screen);
> +    c->setGeometry(popupLocation);

popupGeometry would be a better name. Also, we probably don't need this variable at all.

REPOSITORY
  R108 KWin

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

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