[Differential] [Commented On] D2627: Set the restore geometry after placing a ShellClient for the first time

[graesslin (Martin Gräßlin)]

graesslin added a comment.


  In https://phabricator.kde.org/D2627#48943, @luebking wrote:
  
  > Fixwise good, APIwise, maybe add AbstractClient::placeIn(QRect &area) { Placement::self()->place(this, area); setGeometryRestore(geometry()); } to
  >  a) prevent uninformed messing around with the restore geomery
  >  b) guarantee sync behavior (when placing occurs)
  >
  > ?
  
  
  yep makes sense.

REPOSITORY
  rKWIN KWin

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland, bshah
Cc: luebking, plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160830/b0f08df6/attachment.html>