Scrolling flickers when using virtual scrolling about ka-table

[komarovalexander]

Thanks for reporting this issue!
Fixed in 3.2.2
example: https://stackblitz.com/edit/table-many-rows-ts-nrsdv7?file=Demo.tsx

[yarokon]

Hi, @komarovalexander. This issue is reproducible if the row height is different from another row height. Here is a link with a reproducible example: https://codesandbox.io/s/great-almeida-ojwhc?file=/src/Table.js

[yarokon]

I have modified the example to be closer to a real-world example: https://codesandbox.io/s/great-almeida-ojwhc?file=/src/Table.js

How should I measure row height in this case?

It would be more correct if the table did these calculations, like here: react-windowed-select. Ideally, users using this table shouldn't have to think about how to perform this calculation, because the table has all the data to do this. But if it is not supported, the doc must indicate that additional calculation is required for this case.

[komarovalexander]

Thanks for the feedback!

I will keep this issue open and will investigate it better coming weeks and let you know if any updates.
Really appreciate your use cases and all feedback you provided!

[yarokon]

Please upgrade to version 2.0.4 or higher. At the moment the latest version is 3.0.0

[komarovalexander]

reproduced https://stackblitz.com/edit/table-many-rows-ts-rbc6s8?file=Demo.tsx

[yarokon]

Thanks for the quick fix

[komarovalexander]

Hi @yarokon

By default, ka-table uses equal height for all rows based on height of first row.

But to override this behavior you can use virtualScrolling.itemHeight function to specify each row height
Demo: https://codesandbox.io/s/lucid-herschel-lwmxw?file=/src/Table.js

[komarovalexander]

@yarokon how many dynamic items do you want to render with virtual scrolling >1k, > 10k, 100k, or more?

btw, regarding react-windowed-select are you sure it works well with dynamic items? https://codesandbox.io/s/react-windowed-select-custom-filter-and-component-forked-nf2vt?file=/src/index.js do you know if I miss something in this demo?

[komarovalexander]

Hi @yarokon ,
please update ka-table to 6.7.1 and use code from this demo: https://komarovalexander.github.io/ka-table/#/many-rows-dynamic