#14927 closed bug (fixed)

Opened March 01, 2016 05:09PM UTC

Closed April 12, 2016 07:40PM UTC

Last modified April 12, 2016 07:40PM UTC

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.

Attachments (0)
Change History (2)

Changed April 12, 2016 07:40PM UTC by scottgonzalez comment:1

owner: → scottgonzalez
resolution: → fixed
status: new → closed

In [changeset:"c19fc10e8493007af4617597f0afea5f24860955" c19fc10]:

#!CommitTicketReference repository="" revision="c19fc10e8493007af4617597f0afea5f24860955"
Sortable: Fix incorrect top containment for document

Fixes #14927
Closes gh-1695

Changed April 12, 2016 07:40PM UTC by scottgonzalez comment:2

milestone: none → 1.12.0