graesslin added a comment.
> updateCursor now swaps m_cursorIndex which is a signifcant behaviour change from the old separate setCursor/updateCursor Yes, I did it on purpose as I found the code usage weird and difficult to follow. I thought it's much easier to understand if we go for swapping the index whenever we paint into it. I didn't think of the update followed by update case. So this clearly needs fixing - meh :-( REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D8664 To: graesslin, #kwin, #plasma, davidedmundson Cc: davidedmundson, plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart