Sortable: When options.axis is "x" or "y", allow drag to rearrange placeholder element in connected lists, even when outside of the container element. #650

Closed
wants to merge 1 commit into
from
Jump to file or symbol
Failed to load files and symbols.
+1 −1
Diff settings

Always

Just for now

View
@@ -448,7 +448,7 @@ $.widget("ui.sortable", $.ui.mouse, {
var dyClick = this.offset.click.top,
dxClick = this.offset.click.left;
- var isOverElement = (y1 + dyClick) > t && (y1 + dyClick) < b && (x1 + dxClick) > l && (x1 + dxClick) < r;
+ var isOverElement = ((this.options.axis === 'x') || ((y1 + dyClick) > t && (y1 + dyClick) < b)) && ((this.options.axis === 'y') || ((x1 + dxClick) > l && (x1 + dxClick) < r));
if( this.options.tolerance == "pointer"
|| this.options.forcePointerForContainers