[FEATURE]: Option not to disable TableHeadRow when there's no data

[philipnery]

Feature Check list

• Agree to the Code of Conduct
• Read the README to ensure the feature is not already present
• You read Creating Issues, Features and Pull Requests
• Considered the value versus complexity for all users of the library as well as library maintenance
• Considered if this can be a storybook or documentation example

Is your feature request related to a problem? Please describe

We have buttons placed inside table row headers.

Unfortunately, if the table has no data (and we're using persistTableHead BTW), it gets disabled so we can't anymore click the buttons inside.

Describe the solution you'd like

Either we add another props for this or modify TableHeadRow's disabled when persistTableHead is true.

Maybe something like this?

                  <TableHeadRow
                    className="rdt_TableHeadRow"
                    role="row"
                    dense={dense}
                    disabled={progressPending || (data.length === 0 && !persistTableHead)}
                  >

Thanks,
Philip

[jbetancur]

Thanks for raising this. I've not has as much time as I'd like to work but I will try to get to it the next week. PR's are welcome if you'd like to contribute and get a fix in sooner.

[jbetancur]

Can you share code or preferable a codesandbox https://codesandbox.io/embed/react-data-table-sandbox-ccyuu of how you are adding elements to TableHeadRow?

TableHeadRow is meant to be internally controlled.

In any case, I think the disabled can be moved to TableCol ColumnSortable since it is the sort button that should be disabled or hidden if there is nothing to sort.

[mathieupetrini]

Thanks for your library.
Any updates on this ?

[jbetancur]

Implemented this in RDT next branch/alpha-6.
#776