Similar to patch: 12014 - BUT with some additional changes (see attached diff) and without allowing configuration of tolerance (I prefer allowing configuration)
I don't know if this fixes all of the issues with the above mentioned patch, but simply adding tolerance will cause some behavior problems.
This diff also tracks:
currentTarget - the target over which the drag started. This avoids the drift problem introduced by adding tolerance due to the current code relying on '$(e.target).closest'.
startX & startY - stores and calculates the distance the mouse has moved and only triggers drag start when the tolerance has been exceeded.
dragStart - modified to use currentTarget rather than closest target to avoid the drift issue.
I don't know if this diff fixes all remaining issues, but these were the ones I had encountered during my testing.
Similar to patch: 12014 - BUT with some additional changes (see attached diff) and without allowing configuration of tolerance (I prefer allowing configuration) I don't know if this fixes all of the issues with the above mentioned patch, but simply adding tolerance will cause some behavior problems. This diff also tracks:
currentTarget - the target over which the drag started. This avoids the drift problem introduced by adding tolerance due to the current code relying on '$(e.target).closest'. startX & startY - stores and calculates the distance the mouse has moved and only triggers drag start when the tolerance has been exceeded. dragStart - modified to use currentTarget rather than closest target to avoid the drift issue. I don't know if this diff fixes all remaining issues, but these were the ones I had encountered during my testing.
Attachments
jquery.dragsort.diff
This work item was migrated from CodePlex
CodePlex work item ID: '32717' Vote count: '1'