[DataGrid] Add density change event about mui-x HOT 4 CLOSED

@aress31 Thank you for using MUI X Data Grid and providing useful feedback. 😊
The request you made makes sense and is already made previously in #5647
Fortunately it's already in our backlog and should be prioritized as some bandwidth is available.
Please feel free to checkout the discussion or hop in with your feedback on the mentioned ticket.

I'll close this one just to keep all the conversation on the same place.
Thank you.

from mui-x.

I would suggest that you keep listening to the state changes but only trigger your logic when specific parts of it change. This is flexible enough to cover all use cases, and is exactly what we do inside the grid to trigger targetted re-renders.

// Select the parts of the state you care about
// NOTE: make sure to use the public selectors from the docs, the state's inner structure isn't public
const selector = state => [state.a, state.b, state.c]

const cachedValues = useRef([])
useEffect(() => {
  return apiRef.current.subscribeEvent('stateChange', state => {
    const values = selector(state)
    if (!values.every((v, i) => v === cachedValues.current[i])) {
      // [insert save state code here]
      cachedValues.current = values
    } 
  })
}, [])

I don't like the idea of adding a prop for this specifically, it's too specific and doesn't have a significant advantage over using the implementation above. Does that solve your issue?

from mui-x.

From an ease of use and DX perspective I would still prefer the onDensityChange handler to remain consistent with all the other handlers.

from mui-x.

@mui/xgrid Any other opinions on adding a prop here?

from mui-x.