Next row is added at the wrong postion in a pageable collection when deleting a row in the view

[jurgenkalverboer]

When I remove the highlighted row in the image below.

The next row in the collection should be added at the bottom of the pageable collection. However it is inserted one position too high.

On the pageable collection I call the function remove with the corresponding model as its parameter. In backbone-pageable.js the collectionEventHandler is triggered by this action (with a "remove" event). Line 453 of this file will add a new row to the page collection:

if (nextModel = fullCol.at(pageEnd)) pageCol.push(nextModel);

This function will call the function inserRow immediatly on the pageable collection (in backgrid,js), this thus happens before the remove event will call the function removeRow. The index that is determined by the insertRow function is thus not correct:

var index = collection.indexOf(model);

[excentris]

+1

[oosterholt]

+1

[ericbeijer]

+1

[KramerCJ]

+1

[WRidder]

+1

[Wietjerr]

+1

[beatle01]

+1

[wyuenho]

Let me know if b2a1aa1 fixes this issue. I'll commit a test soon.

[wyuenho]

Actually forget that. It doesn't. Let me fix this in a moment.