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 comment:1
| component: | ui.droppable → ui.sortable |
|---|
Changed February 13, 2012 02:15PM UTC by comment:2
| component: | ui.sortable → [meta] ui.demos |
|---|---|
| owner: | → rdworth |
| status: | new → assigned |
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 comment:3
| resolution: | → fixed |
|---|---|
| status: | assigned → closed |
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 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 comment:5
| milestone: | 1.9 → 1.8.18 |
|---|