[DataGridPro] Infinite loading onRowsScrollEnd intermittent issue not firing

[eversadi]

Latest version

• I have tested the latest version

Steps to reproduce

Link to live example: (required)

https://codesandbox.io/s/s9zvh2?file=/src/Demo.js

video here

• https://www.awesomescreenshot.com/video/30225828?key=db66a35858cb4289fb5b9a044144afb0

Steps:

1. just scroll down to the end of the scroll load the data from the mui example

Current behavior

Data Grid onRowsScrollEnd event does not fire , it just stop at the bottom

Expected behavior

Data Grid onRowsScrollEnd should always fire once reach the end of the scrollbar

Context

I wan't to have a table that can loads a data when the scroll position is at the bottom
icy-paper-s9zvh2 - CodeSandbox.webm

Your environment

npx @mui/envinfo

  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Search keywords: onRowsScrollEnd
Order ID: eversadi-1

[arminmeh]

Can confirm on our own example as well.

Thanks for reporting @eversadi

[lukedukeus]

Just ran into this issue. I found that using disableVirtualization got rid of it, but then everything renders much slower. I found that I was also much more likely to run into this if I had complex logic for my renderCell method.

[arminmeh]

thanks @lukedukeus
this helps reproduce the issue

[github-actions]

This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue.
Now, if you have additional information related to this issue or things that could help future readers, feel free to leave a comment.

Note

We value your feedback @eversadi! How was your experience with our support team?
We'd love to hear your thoughts in this brief Support Satisfaction survey. Your insights help us improve!

[kfowler1337]

How do I know when this is released? Hit the same issue @arminmeh

[arminmeh]

@kfowler1337
It has been released in 7.19.0
Are you using that version?

[efotin]

Will it be released for v6 premium?

[arminmeh]

@efotin
Infinite loading was re-worked for version 7
so the change made here cannot be ported back.
since it is not a regression or a security issue, there is no plan to fix this on v6