rniemeyer / knockout-sortable

A Knockout.js binding to connect observableArrays with jQuery UI sortable functionality
MIT License
547 stars 128 forks source link

afterMove works very sporadically and sometimes not at all #148

Open ivaluev opened 9 years ago

ivaluev commented 9 years ago

as a test pls see this fiddle http://jsfiddle.net/bxn0fsoz/10/

if you move an item first time - alert fires ok - then if second it may fire and may not - try sorting up to 10 times different items - to see how it fires and then stops firing and then starts firing again and again stops

jhasselkus commented 8 years ago

It looks like knockout-sortable isn't compatible with any variety of deferred updates. If you remove the .extend({rateLimit: 50}) from model.items your fiddle works fine.

ricardobrandao commented 8 years ago

+1 for

It looks like knockout-sortable isn't compatible with any variety of deferred updates.

afterMove only runs once per element when using knockout 3.4 with ko.options.deferUpdates = true - please check this fiddle.

@rniemeyer is this a known issue?

166 solved the problem with the deferred updates in the new knockout version.