<div>graesslin added inline comments.</div><br /><div><strong>INLINE COMMENTS</strong><div><div>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.<br />
<br />
My suggestions would be adding a:<br />
    Cursor::setPos(geometry()->center());<br />
<br />
It might also need a test case like the one for netMoveRequest.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKWIN KWin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1434" rel="noreferrer">https://phabricator.kde.org/D1434</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>hein, graesslin<br /><strong>Cc: </strong>plasma-devel, sebas<br /></div>