glideapps / glide-data-grid

🚀 Glide Data Grid is a no compromise, outrageously react fast data grid with rich rendering, first class accessibility, and full TypeScript support.
https://grid.glideapps.com
MIT License
4.07k stars 301 forks source link

Maximum update depth exceed error #941

Open lukasmasuch opened 7 months ago

lukasmasuch commented 7 months ago

TheMaximum update depth exceeded error in InfiniteScroller seems to be back :( This was discussed in the 6.0.0 PR: https://github.com/glideapps/glide-data-grid/pull/810#issuecomment-1864715827 and seemed to have been fixed, but it still sometimes happens. Unfortunately, I'm not able to trigger/reproduce this reliably. This is most likely related to the grow column feature.

image

The stack related to this issue:

https://github.com/glideapps/glide-data-grid/blob/3a97523f632d5cab1e99017e329971a7993aff34/packages/core/src/common/utils.tsx#L242

https://github.com/glideapps/glide-data-grid/blob/17f800d575eedd8e41593ff14e027891faf57a8f/packages/core/src/data-editor/data-editor.tsx#L2478

https://github.com/glideapps/glide-data-grid/blob/17f800d575eedd8e41593ff14e027891faf57a8f/packages/core/src/internal/scrolling-data-grid/scrolling-data-grid.tsx#L217

https://github.com/glideapps/glide-data-grid/blob/17f800d575eedd8e41593ff14e027891faf57a8f/packages/core/src/internal/scrolling-data-grid/scrolling-data-grid.tsx#L240

https://github.com/glideapps/glide-data-grid/blob/17f800d575eedd8e41593ff14e027891faf57a8f/packages/core/src/internal/scrolling-data-grid/infinite-scroller.tsx#L219

https://github.com/glideapps/glide-data-grid/blob/17f800d575eedd8e41593ff14e027891faf57a8f/packages/core/src/internal/scrolling-data-grid/infinite-scroller.tsx#L241