Skip to main content

Search and Top Navigation

#9147 open bug ()

Opened March 08, 2013 10:03PM UTC

Last modified March 23, 2013 11:13AM UTC

Sortable: Write Unit Tests Covering #8572, #8573, #8574

Reported by: mikesherov Owned by:
Priority: minor Milestone: 2.0.0
Component: ui.sortable Version: 1.10.1
Keywords: Cc:
Blocked by: Blocking:
Description

#8572, #8573, #8574 was complex change that broke floating sortables. The floating issue is now fixed and covered, but there's nothing preventing these from regressing.

Here's the fiddles to use as the basis for tests:

#8572:

http://jsfiddle.net/Zvvpt/15/

1. drag the first item and move slowly down, when getting to the bottom of the sortable, the placeholder moves to the top of the sortable

2. drag the second item and move slowly down, the placeholder doesn't move until the item is dragged out of the sortable, where the placeholder jumps quickly to the bottom of the sortable and then goes to the middle again

#8573:

http://jsfiddle.net/6bsGx/3/

1. Click near the bottom of an item and move downwards, notice that the item can't be dragged out of the inner sortable

2. Click near the top of an item and move upward, notice that the item can't be dragged out of the inner sortable

#8574

http://jsfiddle.net/k4nn3/3/

1. Drag an item down, notice that it's hard to put the item between the two inner sortables.

2. If you clicked the top of an item and drag it down , it can be put between the inner sortables, but you can never make it there when moving upwards.

Attachments (0)
Change History (2)

Changed March 08, 2013 10:18PM UTC by mikesherov comment:1

milestone: none2.0.0

Changed March 23, 2013 11:13AM UTC by tj.vantoll comment:2

status: newopen