doesn't work properly with virtual element on ko 2.2

[rniemeyer]

doesn't work properly with virtual element on ko 2.2

[rniemeyer]

@kazrac - the sortable plugin expects that the sortable items will be contained in a single element. If you wrap your template in a div, then it would work properly. Sample: http://jsfiddle.net/rniemeyer/2qp4V/

Does that work for you or maybe you can explain your scenario a bit more, so I can see how to make it work properly?

[kazrac]

hi, thanks for quick answer.
In project I use table and table rows instead of divs. There is one row for main information and one row for details. Details row can by expanded by clicking + in main row. It looks something like this http://jsfiddle.net/KSSDX/2/ . We tried to place 'if' binding inside second details tr but then sorting doesn't work properly in IE8. It is logical to have single contained element for sortable elements. In this case we prevent sorting when at least one row is expanded. Since it worked in older ko version it would be nice to have this functionality in new too :)

[bhandfast]

Similar issue here: http://jsfiddle.net/benzor/3qQ4q/

It works in IE9, but not in IE8. Seems to be caused by fixUpNodesToBeMovedOrRemoved returning undefined. Not sure why but if all whitespace in the template is removed, it works in IE8 too.

[mbest]

The cause is this bug in Knockout, knockout/knockout#498, which means that the afterRender callback gets stale data.

So why is there an error in 2.2 and not in 2.1? That's because the if binding saves the template differently in 2.2 than it does in 2.1. In 2.1, the nodes with if are saved as a template (original nodes are moved) and attached to a div. In 2.2, the nodes are copied and the original nodes are removed (so there's no parent). The error occurs because the sortable code assumes the nodes have a parent.