Column: PT not getting applied for headerCheckbox

[skj-skj]

Describe the bug

according to the docs for Column PT
https://primereact.org/datatable/#pt.column.options

headerCheckbox is available to pass props to Checkbox component but props are not getting applied.

I saw there was similar issue which was fixed #6978 but the file change were in DataTable which doesn't have headerCheckbox pt as per docs

Reproducer

Stackblitz Project with headerCheckbox issue

System Information

PrimeReact: 10.8.2
React: 18.3.1
Vite: 5.4.0

Steps to reproduce the behavior

No response

Expected behavior

props should passed to the checkbox component of headerCheckbox

[github-actions]

Please fork the Stackblitz project and create a case demonstrating your bug report. This issue will be closed if no activities in 20 days.

[melloware]

cc @peconomou929 since we just fixed this can you help out @skj-skj ?

[skj-skj]

Hi @melloware,

I created a Stackblitz project for this issue. In this project, I updated the style of the checkboxes using PrimeFlex classes.

In the CheckboxTablePT component, the rowCheckbox PT works fine as the styles are applied correctly. but the headerCheckbox PT is not working as expected, and the styles are not being applied.

Click on checkbox on the table header to view the behaviour.

[melloware]

Thanks I will take a look.

[peconomou929]

I agree that this is a bug, and what you demonstrate is indeed unexpected behaviour.

However, another way to achieve what you want is to use the "column" entry in the DataTablePassThroughOptions, instead of setting the ColumnPassThroughOptions on the Column itself. That usage is precisely what got fixed in #6979 and #6939.

[melloware]

Oh i see there is a headerCheckbox on the DT and one on the Column. You are right that is confusing.

[skj-skj]

@peconomou929 Thanks that workaround works.
I have updated the Stackblitz reproducer project and added the workaround.

this pattern makes sense as in most cases a DataTable will have only one column as checkbox. so instead of pt individual column for headerCheckbox and rowCheckbox it make sense to pt only DataTable.

[melloware]

yeah now i am wondering if headerCheckbox should be removed from the Column definition.

[melloware]

OK PR submitted

[skj-skj]

@melloware thanks for taking care of this issue.