Opened 13 months ago

Closed 12 months ago

Last modified 12 months ago

#14927 closed bug (fixed)

Regression: containment: "document" broken on sortable

Reported by: zerocrates Owned by: scottgonzalez
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 12 months ago by scottgonzalez

  • Owner set to scottgonzalez
  • Resolution set to fixed
  • Status changed from new to closed

In c19fc10:

Sortable: Fix incorrect top containment for document

Fixes #14927
Closes gh-1695

comment:2 Changed 12 months ago by scottgonzalez

  • Milestone changed from none to 1.12.0
Note: See TracTickets for help on using tickets.