Show / hide all buttons don't set header group visibility state

[ThisIsHG]

This PR is related to #1013

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
material-react-table	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 28, 2024 9:13am
material-react-table-storybook	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 28, 2024 9:13am

[KevinVandy]

I'm actually thinking of reverting all this new logic in MRT. I think that TanStack Table's APIs should handle this.

Though can you tell me what the use case is for seeing whether or not column groups are visible?

[ThisIsHG]

@KevinVandy
We have a table with repeating column groups with the following leaf columns:

Evaluation	Detail A	Detail B	Detail ...

for those where a evaluation is present only the evaluation should be visible by default and for those without the details should be always visible.

We also implemented a detail switch wich shows those columns where a evaluation is present.

But when a column group is hidden as a whole and the detail view is enabled those column groups which should remain hidden appear again because there is no other way of determining if those columns should be set to visible again since we can't rely on the Evaluation column.

[KevinVandy]

@ThisIsHG I think I'll just change TanStack Table like this:

[KevinVandy]

@ThisIsHG If v2.12.1 works for you, we can close this PR.

[ThisIsHG]

Will validate it first thing in morning tomorrow! But thanks in advance!

[ThisIsHG]

@KevinVandy works like a charm!

Thanks for bringing it so fast into the Table!