[DataGrid] The grid not scrollable after shift select multiple rows

[michael-land]

Duplicates

• I have searched the existing issues

Latest version

• I have tested the latest version

Current behavior 😯

No response

Expected behavior 🤔

No response

Steps to reproduce 🕹

Steps:

1.https://mui.com/components/data-grid/demo/
2.shift select multiple rows
3.scroll the grid

Context 🔦

v 5.2.0

Your environment 🌎

`npx @mui/envinfo`

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

Order ID 💳 (optional)

No response

[alexfauquette]

Hi @xiaoyu-tamu I do not manage to reproduce your bug. Please, could you do a recording, or at least provide a precise list of actions such as "click here, pressed this key, ..."

The browser you're using can also be useful to catch such a bug

Screencast.from.13-12-2021.18.04.26.webm.mp4

[michael-land]

Sorry about that

Screen.Recording.2021-12-13.at.12.28.40.mov

Chrome Version 96.0.4664.93 (Official Build) (arm64)
OS: macOS Monterey 12.0.1

Hope this help

[michael-land]

not sure if this is related to checkbox focus.

#3382
#3383

[michael-land]

Hi @xiaoyu-tamu I do not manage to reproduce your bug. Please, could you do a recording, or at least provide a precise list of actions such as "click here, pressed this key, ..."

The browser you're using can also be useful to catch such a bug

Screencast.from.13-12-2021.18.04.26.webm.mp4

I don't have a linux, can you try to scroll down the grid 5~10 rows and try the above steps?

[m4theushw]

@oliviertassinari and @DanailH could you try to reproduce the bug on macOS? I can't on Windows.

[alexfauquette]

Thanks @xiaoyu-tamu

I can reproduce it on ubuntu, To do so I have to scroll slowly, and far enough to make the focus cell disappear. I will have a look

[alexfauquette]

@m4theushw The problems seem to come from virtualization. In getRows(), the array index is used as a key. Then, when the range of rows is modified, it gets a new key and so React considers it as a new component. Leading to a new focus event that triggers the scroll when the cell is hidden.

Is there a reason for not using rowId as a key?