[Differential] [Commented On] D1434: Hook up movable and resizable states.

graesslin (Martin Gräßlin) noreply at phabricator.kde.org
Tue Apr 19 11:55:48 UTC 2016


graesslin added inline comments.

INLINE COMMENTS
  abstract_client.cpp:696-705 Cursor::pos() seems wrong to me. If it's triggered by e.g. the task manager the cursor will be on the task item and thus the first movement will make the window jump.
  
  My suggestions would be adding a:
      Cursor::setPos(geometry()->center());
  
  It might also need a test case like the one for netMoveRequest.

REPOSITORY
  rKWIN KWin

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

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

To: hein, graesslin
Cc: plasma-devel, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160419/30f4adb9/attachment.html>


More information about the Plasma-devel mailing list