[data grid] Disable drag event handlers when row or column reorder are disabled

[DanailH]

Fixes #4841

Codesandbox: https://codesandbox.io/s/billowing-moon-4dw6h8

[mui-bot]

These are the results for the performance tests:

Test case	Unit	Min	Max	Median	Mean	σ
Filter 100k rows	ms	254.3	546.6	470	421.26	124.127
Sort 100k rows	ms	496.9	914.1	838.3	789.58	151.311
Select 100k rows	ms	153	247.9	185	197.1	40.203
Deselect 100k rows	ms	119	241.5	146.4	175.08	49.263

Generated by 🚫 dangerJS against 56a6dc7

[cherniavskii]

Nice!

[m4theushw]

Since the issue in #4841 was already fixed in the latest release according to #4841 (comment), is there another CodeSandbox we can see the bug here, or a test case?

Based on the title of this PR, if we want to disable all event handlers, then we must disable them in the cell too:

mui-x/packages/grid/x-data-grid/src/components/cell/GridCell.tsx

Line 249 in f90a012

onDragEnter={publish('cellDragEnter', onDragEnter)}

These event handlers are used to allow to drop the column header inside any cell of the column.

[DanailH]

Since the issue in #4841 was already fixed in the latest release according to #4841 (comment), is there another CodeSandbox we can see the bug here, or a test case?

Based on the title of this PR, if we want to disable all event handlers, then we must disable them in the cell too:

mui-x/packages/grid/x-data-grid/src/components/cell/GridCell.tsx

Line 249 in f90a012

onDragEnter={publish('cellDragEnter', onDragEnter)}

These event handlers are used to allow to drop the column header inside any cell of the column.

@m4theushw I agree that it's a good idea to remove those as well - I forgot about them. The check should be in the GridRow component probably to avoid checks on each cell.
Regarding the bug - this is just an optimization.