Bug Tracker

Modify

Ticket #2534 (closed bug: fixed)

Opened 5 years ago

Last modified 5 years ago

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:
Blocking: Blocked by:

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

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

Change History

comment:1 Changed 5 years ago by karibouk

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

comment:2 Changed 5 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 5 years ago by karibouk

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

Changed 5 years ago by karibouk

[ui.sortable] Nested sortable bug

comment:4 Changed 5 years ago by paul

  • Status changed from new to closed
  • Resolution set to fixed

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

comment:5 Changed 5 years ago by karibouk

  • Status changed from closed to reopened
  • Resolution fixed deleted

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

comment:6 Changed 5 years ago by paul

  • Status changed from reopened to closed
  • Resolution set to fixed

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 5 years ago by karibouk

  • Status changed from closed to reopened
  • Resolution fixed deleted

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 5 years ago by paul

  • Status changed from reopened to closed
  • Resolution set to fixed

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!

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.