table icon indicating copy to clipboard operation
table copied to clipboard

Column visibility APIs do not work with column groups

Open ashacs opened this issue 1 year ago • 5 comments

TanStack Table version

v8.15.2

Framework/Library version

React v8.15.2

Describe the bug and the steps to reproduce it

When using column helpers to create a column group, consumers are able to provide enableHiding: true as a column def option. However:

  1. column.getToggleVisibilityHandler() or column.toggleVisibility always updates the table's columnVisibility state to have a false entry for that column group's id
  2. column.getIsVisible() always returns true for that column group

Ideally, the getToggleVisibilityHandler, toggleVisibility, getIsVisible, and related HeaderGroup APIs could all be updated to support visibility toggling on column groups (which in turn would show or hide the children columns of that column group). Worst case, if toggling visibility of groups isn't something that can be supported, the enableHiding column def option could be removed from the column helpers API so it's not misleading to consumers.

Your Minimal, Reproducible Example - (Sandbox Highly Recommended)

https://codesandbox.io/p/devbox/dry-river-9t7pg9

Screenshots or Videos (Optional)

This video uses the linked Code Sandbox example which is almost identical to the example in the documentation, but instead of rendering the column leaves as visibility checkbox options it renders the column groups as options. When interacting with the checkbox options which should toggle the column visibility, the visibility state is always set to false and the checkbox is always checked.

https://github.com/TanStack/table/assets/10733340/f5a8fd29-0d71-494a-8190-a8588c1e6ccf

Do you intend to try to help solve this bug with your own PR?

None

Terms & Code of Conduct

  • [X] I agree to follow this project's Code of Conduct
  • [X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

ashacs avatar Apr 18 '24 22:04 ashacs

This still appears to be the case for 8.17.3

wbolduc avatar Jun 20 '24 16:06 wbolduc