[DataGrid] onColumnVisibilityChange doesn't fire when using show all columns

[lovesinatra]

Duplicates

• I have searched the existing issues

Latest version

• I have tested the latest version

Current Behavior 😯

When clicking on "show all" from the columns menu the onColumnVisibilityChange event doesn't fire off

__

Expected Behavior 🤔

onColumnVisibilityChange should be called

Steps to Reproduce 🕹

Steps:

Context 🔦

I'm trying to persist column changes using this event handler

Order id 💳

27855

[joserodolfofreitas]

I'm reopening the issue because it's indeed expected that the callback is called when clicking "show all" or "hide all".
for reproduction, please check the codesandbox example.

[flaviendelangle]

Duplicated #2198

See the conversation for context
We won't fix this callback which is not compatible with a batched visibility switch since it takes a single ID as an argument.
But we introduced a new onColumnVisibilityModelChange which contains all the information necessary.