Hi,
Thanks for your work on this. I was doing a little testing of how very long lists are handled.
I noticed that in the example Fixed Sized Grid in code sandbox it will only handle a row count up to 958698. Any value greater than that and it does not display rows beyond row 958698 (index 958697). I am not surprised there is some upper limit just due to memory and variable maximums, just wondering if this level was surprising and if one can estimate the upper limit of rows/cells.
Thanks.
Did a search over in react-virtualized and I think this issue https://github.com/bvaughn/react-virtualized/issues/193#issue-149169529 is the same question and I see its related to maximum number of pixels supported by the browser...so I'll close this. Please reopen if you think this should be different in react-window. Sorry I didn't look in react-virtualized first.
Hey @delateurj! The issue you linked to (bvaughn/react-virtualized/issues/193) is definitely related. Browsers have a maximum pixel size that they support. For newer browsers, that pixel limit is really high.
In react-virtualized, I implemented a workaround for this (ScalingCellSizeAndPositionManager) but abstractions add cost both in terms of maintenance and runtime performance, and I decided _against_ adding a similar workaround in react-window.
My main reasons for this decision were:
Hope this explanation makes sense and is useful to others who come across this issue in the future.
Thanks. Decision/explanation makes sense...was exploring the edges.
Most helpful comment
Hey @delateurj! The issue you linked to (bvaughn/react-virtualized/issues/193) is definitely related. Browsers have a maximum pixel size that they support. For newer browsers, that pixel limit is really high.
In react-virtualized, I implemented a workaround for this (
ScalingCellSizeAndPositionManager) but abstractions add cost both in terms of maintenance and runtime performance, and I decided _against_ adding a similar workaround in react-window.My main reasons for this decision were:
Hope this explanation makes sense and is useful to others who come across this issue in the future.