Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[question][data grid] data grid don't trigger onRowScrollEnd on infinite loading with less rows than viewport #9022

Closed
2 tasks done
noghartt opened this issue May 17, 2023 · 2 comments · Fixed by #8672
Closed
2 tasks done
Labels
component: data grid This is the name of the generic UI component, not the React module! duplicate This issue or pull request already exists plan: Premium Impact at least one Premium user support: commercial Support request from paid users support: question Community support but can be turned into an improvement

Comments

@noghartt
Copy link

Order ID or Support key 💳 (optional)

49216

Duplicates

  • I have searched the existing issues

Latest version

  • I have tested the latest version

The problem in depth 🔍

Using the infinite loading feature, we have some kind of weird behavior:

When we have less rows than the necessary to scroll the viewport, the onRowScrollEnd and the fetch to news rows aren't being triggered. Is there a way, using DataGrid, to trig it even if isn't overflowing the viewport?

Your environment 🌎

  System:
    OS: macOS 13.3.1
  Binaries:
    Node: 18.1.0 - ~/.nvm/versions/node/v18.1.0/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 8.8.0 - ~/.nvm/versions/node/v18.1.0/bin/npm
  Browsers:
    Chrome: 112.0.5615.137
    Edge: Not Found
    Firefox: Not Found
    Safari: 16.4
  npmPackages:
    @emotion/styled:  10.3.0 
    @mui/base:  5.0.0-alpha.128 
    @mui/core-downloads-tracker:  5.12.2 
    @mui/icons-material: 5.11.16 => 5.11.16 
    @mui/lab:  5.0.0-alpha.129 
    @mui/material: ^5.11.10 => 5.12.2 
    @mui/private-theming:  5.12.3 
    @mui/styled-engine-sc: 5.12.0 => 5.11.11 (5.12.0)
    @mui/styles: ^5.11.9 => 5.12.3 
    @mui/system:  5.12.3 
    @mui/types:  7.2.4 
    @mui/utils:  5.12.3 
    @mui/x-data-grid:  5.17.26 
    @mui/x-data-grid-generator:  5.17.26 
    @mui/x-data-grid-premium:  5.17.26 
    @mui/x-data-grid-pro:  5.17.26 
    @mui/x-date-pickers:  5.0.20 
    @mui/x-date-pickers-pro:  5.0.20 
    @mui/x-license-pro:  5.17.12 
    @types/react: ^18.0.28 => 18.2.6 
    react: 18.2.0 => 18.2.0 
    react-dom: 18.2.0 => 18.2.0 
    styled-components:  5.3.10 
    typescript: 5.0.4 => 5.0.4 
@noghartt noghartt added status: waiting for maintainer These issues haven't been looked at yet by a maintainer support: commercial Support request from paid users labels May 17, 2023
@zannager zannager added support: question Community support but can be turned into an improvement component: data grid This is the name of the generic UI component, not the React module! plan: Premium Impact at least one Premium user labels May 18, 2023
@m4theushw m4theushw removed the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label May 18, 2023
@m4theushw
Copy link
Member

Duplicate of #4184

We're working on improving how this prop works in #8672

Please subscribe to #4371

@m4theushw m4theushw added the duplicate This issue or pull request already exists label May 18, 2023
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale May 18, 2023
Copy link

github-actions bot commented Mar 7, 2024

⚠️ This issue has been closed.
If you have a similar problem, please open a new issue and provide details about your specific problem.
If you can provide additional information related to this topic that could help future readers, please feel free to leave a comment.

How did we do @noghartt?
Your experience with our support team matters to us. If you have a moment, please share your thoughts through our brief survey.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: data grid This is the name of the generic UI component, not the React module! duplicate This issue or pull request already exists plan: Premium Impact at least one Premium user support: commercial Support request from paid users support: question Community support but can be turned into an improvement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants