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
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to