[DataGrid] Fix scrollbar position not being updated after scrollToIndexes

[arminmeh]

Fixes #14830

scrollToIndexes eventually ends up here

mui-x/packages/x-data-grid/src/hooks/features/scroll/useGridScroll.ts

Lines 147 to 152 in 5260148

	if (virtualScrollerRef.current && params.left !== undefined && colRef.current) {
	const direction = isRtl ? -1 : 1;
	colRef.current.scrollLeft = params.left;
	virtualScrollerRef.current.scrollLeft = direction * params.left;
	logger.debug(`Scrolling left: ${params.left}`);
	}

virtualScroller gets the update and the scrollbar follows via an event listener

In this scenario, only scrollbar should be updated, since this was already done for the scroller. it looks like that, in some cases, scroller event gets executed first and it locks scrollbar update with the isLocked flag set to true.

I have changed this to work with the last position (it was already used in one event listener). This means that the event listeners will run as long as the elements are not at the right position.

As mentioned in the issue, it was already really hard to reproduce the bug, but I notice that after update, the scrollbar gets rendered and displayed after every click, while before it would update in the background.

Before: https://codesandbox.io/p/sandbox/trusting-night-rcm484
After: https://codesandbox.io/p/sandbox/ecstatic-hertz-94g86f

[mui-bot]

Deploy preview: https://deploy-preview-14877--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against 6ac90b3

[cherniavskii]

@arminmeh Were you able to reproduce this issue? I tried in Chrome and Firefox and the scrolling works as expected.
UPD: I can reproduce it by dragging the scrollbar thumb 👍🏻

[cherniavskii]

I've updated the PR description with Before/After sandboxes. It looks good to me!

[arminmeh]

I've updated the PR description with Before/After sandboxes

Thanks @cherniavskii
I don't have access to the "After" sandbox

[cherniavskii]

Updated!

[cherniavskii]

@arminmeh I just noticed that this PR introduced a regression: https://deploy-preview-14877--material-ui-x.netlify.app/x/react-data-grid/#premium-plan

Screen.Recording.2024-10-08.at.22.09.55.mov

[arminmeh]

I'll take a look

[arminmeh]

#14888