[data grid] columnResizeStop is not emitted when column isn't resized

[lauri865]

Steps to reproduce

Link to live example: (required)

Steps:

1. https://mui.com/x/react-data-grid/
2. Pick the first datagrid and just click on the column resizer
3. After the click, witness how column resizer stays in the active state (brighter highlight in dark mode)

Current behavior

Column resizer gets stuck in the active state

Expected behavior

Column resizer should revert to non-active state.

Context

No response

Your environment

npx @mui/envinfo

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

Search keywords: columnResizeStop

[michelengelen]

Hey @lauri865
Thanks for raising this. I can confirm this bug and there is a simple solution for this:

diff --git a/packages/x-data-grid/src/hooks/features/columnResize/useGridColumnResize.tsx b/packages/x-data-grid/src/hooks/features/columnResize/useGridColumnResize.tsx
index e4d3c51d9..475938741 100644
--- a/packages/x-data-grid/src/hooks/features/columnResize/useGridColumnResize.tsx
+++ b/packages/x-data-grid/src/hooks/features/columnResize/useGridColumnResize.tsx
@@ -422,6 +422,7 @@ export const useGridColumnResize = (
         nativeEvent.clientY === prevClientY
       ) {
         refs.previousMouseClickEvent = undefined;
+        apiRef.current.publishEvent('columnResizeStop', null, nativeEvent);
         return;
       }
     }

BUT: the check for the double click event seems as if it is inefficient. When clicking only once the previous event will be the mousedown event and we are checking basically if the mouseup event is following in under 300 ms.

@mui/xgrid WDYT?

[mateuseap]

Hi! I'd like to know if I can open a PR for this one :)

[romgrk]

Hey! Yes you can take this one.