RowClickFunction could have other returns besides string

I would like to raise a dissussion about the data-grid prop

rowClick?: string | RowClickFunction;

My question is on top of RowClickFunction, why does it always expect a string? I say for the case when the row is not valid when redirecting I could return a void, undefined or false, what do you think?

Why would RowClickFunction always need to return string and for me to keep it on the page I have to add a return ''

[fzaninotto]

As I understand, in your case, a click on a datagrid row may or may not have an action. If this is the case, this should be handled before the rowClick function is called, because the pointer shows that a row is clickable. It would be misleading for end users to indicate with the mouse pointer that a row is clickable, then do nothing on click.

So I suggest adding something like the current isRowSelectable prop to the Datagrid - why not isClickable?

But I don’t think we should accept a false or null response to rowClick.

[fzaninotto]

This is not a use case we want to support out of the box. If you need that UX, you’ll have to pass your own DatagridRow, or a build a custom Datagrid altogether.