Opened 15 months ago

Closed 14 months ago

Last modified 14 months ago

#14927 closed bug (fixed)

Regression: containment: "document" broken on sortable

Reported by: John Flatness Owned by: Scott González
Priority: minor Milestone: 1.12.0
Component: ui.sortable Version: 1.11.4
Keywords: Cc:
Blocked by: Blocking:

Description

"document" containment fails on sortable when the list grows even somewhat large.

See the attached fiddle using UI 1.11.4: dragging the topmost elements seems to work fine, but trying to drag one of the bottom elements causes it to snap upward immediately. Also, it's impossible to drag even the top elements past a certain point in the list. Essentially, "document" is being interpreted as some much smaller (in particular, shorter) containment box.

This does not occur with UI 1.11.2, so it seems likely that this is a regression from 1.11.3 (possibly this change? https://github.com/jquery/jquery-ui/commit/fa460f36dc8191e4fe23c02bcbcc7d5006b51d7e).

Broken fiddle (1.11.4): https://jsfiddle.net/Lbhw3zdc/

Working fiddle (1.11.2): https://jsfiddle.net/Lbhw3zdc/1/

Tested on Chrome and Firefox.

Change History (2)

comment:1 Changed 14 months ago by Scott González

Owner: set to Scott González
Resolution: fixed
Status: newclosed

In c19fc10:

Sortable: Fix incorrect top containment for document

Fixes #14927
Closes gh-1695

comment:2 Changed 14 months ago by Scott González

Milestone: none1.12.0
Note: See TracTickets for help on using tickets.