Move to first page after rows is updated

[cmbankester]

No description provided.

[cmckni3]

Should probably adjust the code to be tailored towards this edge case.

When rows is replaced with a new array that is shorter and you're on a page after the new pages length, the table needs to go to a page of the new dataset.

[cmbankester]

Ah, yeah, I can see that being an issue. I'm worried it'll be strange if you push or remove a record to/from the rows array and it sends you back to the first page.

What if we extended the DataTable API with a addRecord and removeRecord function that doesn't mess with the current page, and a replaceRows function that does?

[cmckni3]

Yeah it would be jarring to have the page automatically change. I like the idea of extending the API.

[cmbankester]

@cmckni3 How's that?

[cmbankester]

There, that should be good. So now, if you want to reset the rows and have it automatically reset to the first page, just call table.replaceRows(array_of_new_rows). And if you want to push/pop rows, you can now call table.addRecord(new_row) and table.removeRecord(current_row)

[cmbankester]

We should probably work on getting some API docs on the github repo / demo page.

[cmckni3]

Looks good