[Differential] [Commented On] D3019: [kstyle] Implement window moving on Wayland

graesslin (Martin Gräßlin) noreply at phabricator.kde.org
Sun Oct 16 15:31:04 UTC 2016


graesslin added inline comments.

INLINE COMMENTS

> hpereiradacosta wrote in breezewindowmanager.cpp:264
> mmm. I know delta functions are handy, but isn't this one a tad complicated ? Personnally I find this code quite hard to read. (especially the sequence of closing }s and )s later on ...

I can change it to be a private method, but it requires to also keep the registry around, although we later on don't need it.

REPOSITORY
  rBREEZE Breeze

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

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

To: graesslin, #plasma_on_wayland, hpereiradacosta
Cc: sebas, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161016/7c9f3429/attachment.html>


More information about the Plasma-devel mailing list