[DataGrid] Using mouseup event for focus affects performance

[romgrk]

After the change in #6489, the grid focus now happens on the mouseup event. Unfortunately, this means that react can't batch the updates that happen during the click event, so it needs to render twice whenever there is a focus change.

mui-x/packages/grid/x-data-grid/src/hooks/features/focus/useGridFocus.ts

Lines 440 to 447 in 463de84

	React.useEffect(() => {
	const doc = ownerDocument(apiRef.current.rootElementRef!.current);
	doc.addEventListener('mouseup', handleDocumentClick);
	return () => {
	doc.removeEventListener('mouseup', handleDocumentClick);
	};
	}, [apiRef, handleDocumentClick]);

We should revert that change and fix the demo issue otherwise.

[romgrk]

cc @cherniavskii

[cherniavskii]

@romgrk I don't think I understand the batching issue here.
I tried both mouseup and click, and the task takes roughly the same time in both cases:

mouseup

click

Am I missing something?

[romgrk]

Closing, stale.

[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.

@romgrk: How did we do? Your experience with our support team matters to us. If you have a moment, please share your thoughts in this short Support Satisfaction survey.