Bug Tracker

Opened 12 years ago

Closed 12 years ago

#2534 closed bug (fixed)

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)

ui.sortable_nested_bug.rar (45.5 KB) - added by karibouk 12 years ago.
[ui.sortable] Nested sortable bug

Download all attachments as: .zip

Change History (9)

comment:1 Changed 12 years ago by karibouk

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

comment:2 Changed 12 years ago by karibouk

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()

comment:3 Changed 12 years ago by karibouk

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

Changed 12 years ago by karibouk

Attachment: ui.sortable_nested_bug.rar added

[ui.sortable] Nested sortable bug

comment:4 Changed 12 years ago by paul

Resolution: fixed
Status: newclosed

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

comment:5 Changed 12 years ago by karibouk

Resolution: fixed
Status: closedreopened

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

comment:6 Changed 12 years ago by paul

Resolution: fixed
Status: reopenedclosed

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.

comment:7 Changed 12 years ago by karibouk

Resolution: fixed
Status: closedreopened

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

comment:8 Changed 12 years ago by paul

Resolution: fixed
Status: reopenedclosed

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!

Note: See TracTickets for help on using tickets.