DataTable: support onClick on DataTableRow

[edoardo]

Passing onClick on DataTableRow is not supported at the moment.
There are cases, like in the OpenFileDialog where it's desirable that the whole row is clickable, regardless of which cell the mouse hits.
I could obtain a similar result by adding the onClick on each cell of the row, but I don't see a reason why not have an optional onClick on the row component.

[Mohammer5]

I think this references the issue that we (it's one of our goals, right? @dhis2/team-platform) want to forward props to the root element / component

[HendrikThePendric]

Yeah, agreed. Ideally, I would like to replace our current Table components with the ones in components/table/src/data-table/table-elements, which already forward props.

[JoakimSM]

I second this request. Would it also make sense to use the pointer-cursor when an onClick event handler is passed? (goes for both the DataTableCell and DataTableRow)

[Mohammer5]

I think that depends on the implementation. If we add an explicit onClick prop, then yes, this would make a lot of sense.
Otherwise, if we just forward props, this should be done by providing a className or style prop

[edoardo]

Is this likely going to make it for 2.37?