lisin added a comment.

  I was reluctant to introduce a new variable with such a broad scope 
(`kickoff.dragStartRow`) but using the `startRow` from `DropArea` would 
sometimes still mess up the order - the item would get the new value of 
`startRow` if the user moved the cursor back inside `DropArea`, so it would be 
snapping back to the wrong position. It's not really something a user would 
commonly do, so I can edit the code to use it instead. Otherwise, `startRow` 
wasn't used anywhere so I removed it completely.

REPOSITORY
  R119 Plasma Desktop

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

To: lisin, #plasma, ngraham, hein
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, 
ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, 
apol, mart

Reply via email to