Sortable: check default intersection by calculating surface overlap. Fixed #3022 and #8342 - incorrect behavior of 'intersect' tolerance option

[ngsilverman]

Fixed http://bugs.jqueryui.com/ticket/3022 and http://bugs.jqueryui.com/ticket/8342
#3022 - handle outside sorting element does not work
#8342 - incorrect behavior of 'intersect' tolerance option

[mikesherov]

@ngsilverman, can you provide some unit tests for these fixes?

[tjvantoll]

#8342 has already been fixed but #3022 is still a problem in master.

[ngsilverman]

@mikesherov, I'll look into it.

@tjvantoll, from what I can see #8342 has not been fixed because it's not a duplicate of #8274. If you remove the axis: 'x' from the fiddle you'll see that when you sort a tab and if the mouse pointer isn't over other tabs, it won't sort regardless of how large the intersection is.

[tjvantoll]

@ngsilverman Good call. I reopened the ticket.

[scottgonzalez]

@ngsilverman Are you still planning on writing tests for this?

[scottgonzalez]

@ngsilverman @mikesherov What's the status of this? If it's not getting any traction, I'd like to close this.

[zakdances]

I'm still seeing this problem in Chrome even when the axis: 'x' option is set. The tolerance setting seems completely wonky regardless of what it's set at...sometimes it will require 100% overlap, other times a 50% overlap, and sometimes it even requires an item to be dragged all the way past the target item.

[jzaefferer]

Closing due to lack of tests and activity. One underlying ticket is still open and has a reference to this, maybe someone can pick this up and write a test.