D26883: Turn AbstractClient::move() into a convenience method

Vlad Zahorodnii noreply at phabricator.kde.org
Thu Jan 23 20:17:21 GMT 2020


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

REVISION SUMMARY
  Maintaining two different ways of updating geometry can be challenging.
  For what it's worth, prior to 5.18, move() didn't move the decoration
  input window because someone forgot to update X11Client::doMove().
  
  This change turns move() into a convenience method to simplify geometry
  handling. It adds a little overhead, but on the other hand move() is
  not used frequently.

REPOSITORY
  R108 KWin

BRANCH
  move-wrap-set-frame-geometry

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

AFFECTED FILES
  abstract_client.cpp
  abstract_client.h
  internal_client.cpp
  internal_client.h
  x11client.cpp
  x11client.h
  xdgshellclient.cpp
  xdgshellclient.h

To: zzag, #kwin
Cc: kwin, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20200123/74a22987/attachment.html>


More information about the kwin mailing list