SlickGrid icon indicating copy to clipboard operation
SlickGrid copied to clipboard

Grid content disappear when dragging scrollbar on big data sets

Open JoeOM opened this issue 7 years ago • 1 comments

Hi

I love SlickGrid, but unfortunately, the content disappears when you drag the scrollbar adjuster with big datasets. This does not happen in a few other grids which make me think it can also be done in SlickGrid.

DevExpress React grid, for example, does not seem to have this issue...

https://devexpress.github.io/devextreme-reactive/react/grid/demos/featured/virtual-scrolling/

Could the community maybe look into this?

Thank you very much guys!

JoeOM avatar Mar 14 '18 05:03 JoeOM

Yeah, I think we've talked about some kind of rate limiting before. I think the events just pile up, and most could be discarded.

The difference between those grids and this one, though, is that they're professionally developed, this one is purely volunteer, and we're all quite busy!

So I'll put this on the 'todo' list. I'm trying to get a web site together for SlickGrid, partly to marshal the community more effectively and make use of those valuable people who might have time to just play with things like this and come up with a viable solution. Sorry I can't do better.

6pac avatar Mar 14 '18 06:03 6pac

@6pac while converting SlickGrid to TypeScript I found that the reason why we always see white section when scrolling large dataset is because of this scroll throttling shown below

https://github.com/6pac/SlickGrid/blob/4b06690a2d5d225537deaec943b408bcd555256c/slick.grid.js#L237

which is used in the action throttle

https://github.com/6pac/SlickGrid/blob/4b06690a2d5d225537deaec943b408bcd555256c/slick.grid.js#L4464-L4468

it looks like you were the person who made this change via this commit https://github.com/6pac/SlickGrid/commit/93f043fa587e377462d2fe0fa7525ff9f1b3f3ef

do you know why we have to use 50 as a throttling? perhaps we could keep the 50 as default but also provide a grid option? with a grid option we could let the user try for themselves. I tried with 0 throttling and it does seem to cause some kind of UI issues or flickering but using 5 seems like a good number, for example below is with a throttling of 5

brave_oFRK7LNJ2D

BTW, this is the new styling theme 😉

versus the hard coded throttling of 50 which we see the white section appears much more and longer, 50ms seems like a long time for the user to wait for re-rendering

brave_RS5yeyRzYT

ghiscoding avatar Jul 11 '23 04:07 ghiscoding

No recollection of doing this at all (and no commend in the commit :-/ ). Happy with a settable delay and a change to the default.

6pac avatar Jul 11 '23 23:07 6pac

ok I'll add it to the TypeScript version only for now, if we ever have other fixes to commit on current version then I could backport it if need be.

ghiscoding avatar Jul 12 '23 00:07 ghiscoding

improvements were made in v5.0, see SlickGrid 5.0 Announcement and Migration

ghiscoding avatar Sep 19 '23 18:09 ghiscoding