Griddle icon indicating copy to clipboard operation
Griddle copied to clipboard

Infinite scroll throws exception with latest chrome

Open tarelli opened this issue 8 years ago • 9 comments

It used to work fine before updating Chrome to the latest, now with Version 56.0.2924.76 there is an exception Maximum Call stack exceeded. Have other people encountered this? Thanks!

screen shot 2017-01-28 at 12 55 19

tarelli avatar Jan 28 '17 12:01 tarelli

Getting the same with latest chrome on 0.6... the error seems to be gone with an upgrade to 0.7.1, but still when infinite scrolling through loads of items griddle gets stuck for a while then eventually recovers but no max stack calls in the console anymore.

gidili avatar Feb 01 '17 11:02 gidili

How many columns do you have during this? I seem to be running into the same issue with the upgrade to 0.7.1. Noticed this issue isn't there when there are ~5 columns with single values that do no span multi lines. As soon there are more columns or cells with massive values; the call stack error appears.

What seems to be happening is that it gets stuck calling the gridScroll() as the values it checks against jumps back and forth so it gets stuck setting the new state with different values since gridScroll() is also in the componentDidUpdate.

I increased the adjustedRowHeight() a little and that seemed to resolve the issue.

bjoe87 avatar Feb 06 '17 13:02 bjoe87

@joellanciaux Could you please look into this one?

ryanlanciaux avatar Feb 07 '17 11:02 ryanlanciaux

@gidili For me what you said is not working

ChandraSekharTiwari avatar Feb 14 '17 18:02 ChandraSekharTiwari

We're having the same problem, you can see demo here - http://codepen.io/rosentalaviv/full/ZLdLLq/ It seems to be related to the variable rows height. It does not occurs you specify rowHeight property and make sure all rows are set to this height.

avivr avatar Feb 19 '17 20:02 avivr

If row heights are different this method doesn't work. Does anybody know is this bug already fixed in 0.8? Is there infinite scroll implementation in griddle 1.2.0?

vozni4iy avatar Mar 30 '17 11:03 vozni4iy

Any progress on this?

bjoe87 avatar May 16 '17 19:05 bjoe87

@ryanlanciaux thinking to attempt an upgrade to 1.x to resolve this (and other infinite scrolling issues with loads of data and/or variable row heights) - can I ask if you have any indication that in more recent version infinite scrolling is more robust? Using pagination right now to get around some issues but would really like to use this feature as it's awesome! 😄

gidili avatar Jun 28 '17 18:06 gidili

check the fork I have; this may have been fixed in mine. Let me know if it works for you @gidili

bjoe87 avatar Jun 28 '17 20:06 bjoe87