Bug Tracker

#2534 closed bug (fixed)

Opened March 17, 2008 10:59AM UTC

Closed April 17, 2008 12:14PM UTC

ui.sortable - imbricate sortable bug

Reported by: karibouk Owned by: paul
Priority: major Milestone: 1.2.4
Component: ui Version: 1.2.3
Keywords: Cc:
Blocked by: Blocking:
Description

important bug in sortable :

a sortable into an another sortable doesnt work

example :

<div class="sortable1">
   <div class="item sortable2">
      <div class="item"></div>
      <div class="item"></div>
   </div>
   <div class="item"></div>
   <div class="item"></div>
</div>

$('.sortable1').sortable()
$('.sortable2').sortable()
Attachments (1)
Change History (8)

Changed March 17, 2008 02:20PM UTC by karibouk comment:1

in fact the patch in Ticket #2417 (closed bug: fixed) doesnt fix anything, an another bug appear

Changed March 18, 2008 04:56PM UTC by karibouk comment:2

in this case too :

<div class="sortable1">
   <div class="item sortable2">
      <div class="sortable2">
      	 <div class="item">----</div>
         <div class="item">----</div>
      </div>
   </div>
   <div class="item">----</div>
   <div class="item">----</div>
</div>
$('.sortable1').sortable()
$('.sortable2').sortable()

Changed March 18, 2008 04:57PM UTC by karibouk comment:3

aem.. without the "sortable2" in "item sortable"

Changed March 25, 2008 02:44PM UTC by paul comment:4

resolution: → fixed
status: new → closed

This should work now in the latest trunk version. Make sure the sortables don't have shared items.

Changed March 26, 2008 09:17AM UTC by karibouk comment:5

resolution: fixed
status: closed → reopened

it still doesnt work in my case, in the ui.sortable_nested_bug file.

Changed April 04, 2008 12:48PM UTC by paul comment:6

resolution: → fixed
status: reopened → closed

This is definitely fixed - In your test file, you are trying to set items that are sortable from both items, since you select all with the class ".item". This of course doesn't work, having two sortables share the same item.

Changed April 07, 2008 12:11PM UTC by karibouk comment:7

resolution: fixed
status: closed → reopened

sorry for re-re-reopening.. but in my example, i used the class ".item" for the first sortable and class ".item2" for the second.. i don't understand

Changed April 17, 2008 12:14PM UTC by paul comment:8

resolution: → fixed
status: reopened → closed

I still don't know why it happended in a old version of UI - some of the other fixes must have also adressed this one.

I tested your exact test file with a new UI version, and it works fine. Please download 1.5b2 from our homepage. Thanks!