Dragging elements causes collapsed branches to expand automatically about nestedsortable

[louisajaca]

I think I have fixed the issue using the new _clearEmpty:

_clearEmpty: function (item) {
            function replaceClass(elem, search, replace, swap) {
                if (swap) {
                    search = [replace, replace = search][0];
                }

                $(elem).removeClass(search).addClass(replace);
            }

            var o = this.options,
                childrenList = $(item).children(o.listType),
                hasChildren = childrenList.is(':not(:empty)');

            var doNotClear =
                o.doNotClear ||
                hasChildren ||
                o.protectRoot && $(item)[0] === this.element[0];

            if (o.isTree) {
                replaceClass(item, o.branchClass, o.leafClass, doNotClear);

                if (doNotClear && hasChildren) {
                    if (childrenList.is(':visible'))
                        replaceClass(item, o.collapsedClass, o.expandedClass);
                    else
                        replaceClass(item, o.expandedClass, o.collapsedClass);
                }
            }

            if (!doNotClear) {
                childrenList.remove();
            }
        }

[MSvelander]

Even with the new _clearEmpty there is an issue.

If I have all items de-collapsed.

• Then collapse 3 (not 3.1 and 3.3)
• Move 4
• De-collapse 3

3.1 and 3.3 is collapsed but with de-collapsed icon.

De-collapse 3.1 and 3.3 and the icon change to collapse-icon.

Any ideas on a solution?

[larikitty]

+1 (same issue)

[noraheuer]

I edited the _clearEmpty (v. 2.1a)
This seems to work for me.

#816 because :empty includes text nodes as child elements
- hasChildren = childrenList.is(':not(:empty)');
+ hasChildren = childrenList.has('li').length;

#825-828 removed as that caused all branches of the tree to open
- if (doNotClear && hasChildren) {
-   replaceClass(item, o.collapsedClass, o.expandedClass);
- }

#832-833 remove expandedClass when childList is removed
- childrenList.remove();
+ childrenList.parent().removeClass(o.expandedClass);
+ childrenList.remove();

complete method

_clearEmpty: function (item) {
    function replaceClass(elem, search, replace, swap) {
        if (swap) {
            search = [replace, replace = search][0];
        }

        $(elem).removeClass(search).addClass(replace);
    }

    var o = this.options,
        childrenList = $(item).children(o.listType),
        hasChildren = childrenList.has('li').length;

    var doNotClear =
        o.doNotClear ||
        hasChildren ||
        o.protectRoot && $(item)[0] === this.element[0];

    if (o.isTree) {
        replaceClass(item, o.branchClass, o.leafClass, doNotClear);             
    }

    if (!doNotClear) {
        childrenList.parent().removeClass(o.expandedClass);
        childrenList.remove();
    }
}