Skip to main content

Search and Top Navigation

#8114 closed bug (fixed)

Opened February 11, 2012 04:13PM UTC

Closed February 13, 2012 02:20PM UTC

Last modified February 13, 2012 02:21PM UTC

Sortable "Connect Lists" demo doesn't allow dropping as last element

Reported by: 2of11 Owned by: rdworth
Priority: minor Milestone: 1.8.18
Component: [meta] ui.demos Version: 1.8.17
Keywords: Cc:
Blocked by: Blocking:
Description

I've reproduced this on Chrome and Safari (most up-to-date versions as of now) on my Mac:

In this example: http://jqueryui.com/demos/sortable/#connect-lists

Take one element from the first list and move it straight down, then across and place it as the last element into the new list. If you do this without hovering over an element in the second list then you will not be able to place that element. However, if you moved over any element in the target list then you can in fact place the element at the bottom of the list.

Given the simplicity of the demos (hard to get something wrong there), I thought it better to put this into the ui.droppable category rather than the ui.demos category, but feel free to move the ticket if I misplaced it.

Attachments (0)
Change History (5)

Changed February 11, 2012 04:19PM UTC by scottgonzalez comment:1

component: ui.droppableui.sortable

Changed February 13, 2012 02:15PM UTC by rdworth comment:2

component: ui.sortable[meta] ui.demos
owner: → rdworth
status: newassigned

This behavior is as designed. When you take an element down and then straight right to place it below item 5, you're not placing it as the last item in the list, you're placing it *below* the list, and because it has yet to be dragged over a connected list, it's still set to drop as the last item in the original list. The difference you've noted is that if the item is first dragged over the container, a space is made for it, because the container is expecting it. The behavior (and logic) is lot easier to see with borders around each sortable container http://jsbin.com/unafef/ . I don't think any change should be made in sortable but the workaround is to add some padding to the bottom of each sortable container.

Given that, looks like this is indeed a bug in the demos, as this demo doesn't have that workaround.

Changed February 13, 2012 02:20PM UTC by Richard D. Worth comment:3

resolution: → fixed
status: assignedclosed

Sortable demos: connect lists - added some bottom padding to each container. Fixed #8114 - Sortable "Connect Lists" demo doesn't allow dropping as last element

Changeset: efdf6cdf9f0e6aaecad7b3ac53afca38938d9048

Changed February 13, 2012 02:20PM UTC by Richard D. Worth comment:4

Sortable demos: connect lists - added some bottom padding to each container. Fixed #8114 - Sortable "Connect Lists" demo doesn't allow dropping as last element

(cherry picked from commit efdf6cdf9f0e6aaecad7b3ac53afca38938d9048)

Changeset: fff9d87674b83c6a07569b6ce240d970b0ef61c2

Changed February 13, 2012 02:21PM UTC by rdworth comment:5

milestone: 1.91.8.18