#168 new
Cyrus

Sortable trees are not working well in the current version

Reported by Cyrus | February 10th, 2009 @ 02:32 PM

I have found other examples of the sortable tree that work pretty good. I was wondering myself why the drag&drop was not working that good in my case.

I finally found out that these other examples are using an older version of scriptaculous.

Please test out yourself:

Old version of scriptaculous - working good: http://www.woltlab.org/sortable1

New version of scriptaculous - working not so good http://www.woltlab.org/sortable2

Sreencast: http://screencast.com/t/1VtzghG8

Full discussion: http://groups.google.com/group/p...

Comments and changes to this ticket

  • Kalou

    Kalou May 12th, 2009 @ 02:24 PM

    Hello, excuse for my poor english, i'm French. I found a correction for dragdrop.js who fixe the bug. In line 689 we find :

    var options_for_tree = {
      onHover:      Sortable.onEmptyHover,
      overlap:      options.overlap,
      containment:  options.containment,
      hoverclass:   options.hoverclass
    };
    
    

    Remplace by :

    var options_for_tree = {
      onHover:      Sortable.onEmptyHover,
      overlap:      options.overlap,
      containment:  options.containment,
      hoverclass:   options.hoverclass,
      tree:         options.tree
    };
    
    

    I joined the file fixed.

  • Cyrus

    Cyrus May 13th, 2009 @ 09:19 AM

    works great -> so it looks more like a bug that this line is missing ...

  • emz452

    emz452 July 13th, 2009 @ 02:40 AM

    • Assigned user cleared.

    Hi there,

    I'm having the same problem of the drag and drop sortable list not being 'fluid' or 'smooth' with the latest version of scriptaculous and firefox. I tried using Cyrus' older version of scriptaculous from his example above and it worked perfectly.

    I'm not using any nesting / trees, just a simple

    • sortable list.

      I've tested in Firefox 3 and IE 7 and it's actually working as expected in IE, but not Firefox.

      Using scriptaculous.js v1.8.1

      I'm wondering if there's a similar fix for non-nested lists to the one posted by Kalou or if anyone else has had the same problem?

      Cheers,
      Em

  • yaz

    yaz May 13th, 2010 @ 06:35 PM

    • Assigned user set to “ronin-15560 (at lighthouseapp)”

    Confirmed on FF 3.5.9, IE 8 & 7. (Windows Vista)

    Also, on the example provided above (http://www.woltlab.org/sortable2/)ou can't nest an item on the last children. Example: you can't drag something under "Features".

    Once I added the line of code provided by Kalou, it was fixed.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

script.aculo.us is an open-source JavaScript framework for visual effects and interface behaviours.
<br/><b>Source available from github</b>
The Git repository resides at:
<a href="http://github.com/madrobby/scriptaculous">http://github.com/madrobby/scriptaculous</a>
<br/>Check out the current development trunk with:
<code>git clone git://github.com/madrobby/scriptaculous.git</code>
<br/>As <b>script.aculo.us 1.xx is feature-frozen</b>, this development trunk is for <b>bugfixes only</b>.
<br/>New development should happen only for
<b>script.aculo.us 2</b>.
<br/><b>Creating a bug report</b>
When creating a bug report, be sure to include as much relevant information as possible. Post a an example that shows off the problem. Preferably, <b>alter the unit tests</b> and show through either changed or added tests how the expected behavior is not occuring.

People watching this ticket

Attachments

Tags

Pages