fix inertia: take care of the case when t is zero

[chuganzy]

This PR aims to fix an issue where Reorder.Item randomly gets stuck as I posted here.

Related:

• [BUG] Reordering randomly stuck #2101
• Fix incorrect Reorder.Item order calculation #2231

As noted here:

requestAnimationFrame timestamps can come through as lower than the startTime

which means that startTime possibly can be equal to currentTime and t is 0 in that case. inertia returns the state with done: true in that case, and stops the animation immediately.

[mattgperry]

Is it possible to add a test on inertia for this behaviour?

[chuganzy]

@mattgperry Thank you for your comment and indeed. Just addressed here. Please let me know!

[ekatzenstein]

Is there an update on this PR? Having issues with reorders as well

[nick-keller]

@ekatzenstein their is this PR #2231 waiting to be merged as well

[davisg123]

Is there anything blocking getting this merged in? It seems like a high impact, edge case fix that includes a test

[nick-keller]

Shouldn't this be merged? It looks like a very simple fix that is tested. Or maybe I am missing something?

[marwhyte]

Is there an update on this? I'm also having similar issues with reordering

[bel0v]

@mattgperry any chance we could have these 2 PRs merged please? The issue has been affecting our production for a while too

[nick-keller]

Seeing the recent activity, I am bringing this PR up again! @mattgperry it is the last brik to fix the re-order component that is broken!

[nick-keller]

Should this PR be merged or closed?