<table><tr><td style="">hein added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D22306">View Revision</a></tr></table><br /><div><div><p>I see what this is trying to do, but some of the details seem a bit wrong.</p>

<p>E.g. you're only setting <tt style="background: #ebebeb; font-size: 13px;">dragging</tt> to false in a code branch that's only executed when it already is false, which means it's not going to be set to false on a drag release, and pressX/Y also won't be unset on a release that doesn't happen above an iten anymore. You can keep the code flow change (the main thing this is probably fixing is that wrong check for the return value of <tt style="background: #ebebeb; font-size: 13px;">updatePositionProperties</tt>), but please rework the patch a bit to make sure that the state of these variables still gets updated in the way they should on a release.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22306">https://phabricator.kde.org/D22306</a></div></div><br /><div><strong>To: </strong>luc4, ngraham, trmdi, Plasma<br /><strong>Cc: </strong>hein, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>