Fix sorting issue with memo and virtualization

[duckboy81]

Problem:
When using memoMode = "row" and table virtualization, when sorting a column, row positioning may not be correctly updated if the row existed (in DOM?) before AND after the sort.

Fix:
Add additional check to propsAreEqual function for Memo_MRT_TableBodyRow to check if the row index has changed.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated
material-react-table	✅ Ready (Inspect)	Visit Preview	💬 Add your feedback	Jan 10, 2023 at 0:45AM (UTC)
material-react-table-storybook	✅ Ready (Inspect)	Visit Preview	💬 Add your feedback	Jan 10, 2023 at 0:45AM (UTC)

[duckboy81]

See issue here: https://codesandbox.io/s/lucid-bessie-vohfsb?file=/src/TS.tsx

EDIT: Sort the first name column a few times and you'll see some discrepancies.

[KevinVandy]

Wonder if we should also add the row id to the list to check

[duckboy81]

Wonder if we should also add the row id to the list to check

I can't say I'm too acquainted with all the inner workings of the project, but it would seem row.id probably wouldn't change unless the entire row object is rebuilt with a change to the data prop.

I think the existing object reference check prev.row === next.row is probably good enough.

Thanks for this project btw, amazing OOTB capabilities.