Table: also pass rowData to rowClassName and rowStyleObject

This already happens for onRowClick. It would be nice if rowClassName could also receive the rowData in its parameters when it’s used as a function.

At first sight it looks like it’s just a matter of moving this line up and passing it to the functions. For the header row it can be left undefined.

I can do a PR if you want.

[bvaughn]

Hey @Janpot

Thanks for the clearly articulated feature request and the offer to submit a PR. (Especially the second one!)

This sort of request comes up fairly often- to support more event types at the row-level or to pass more data to an event handler. At first I added features like this but more recently I’ve started pushing back because I think it has the possibility to cause bloat. (If 99% of users don’t use a feature- and it’s possible to implement in user code- then it probably doesn’t belong in the lib.)

Anyway, this is one such feature that can be implemented in your application code and so I think it’s probably best to stay there.

const list; // Assuming your data lives in an Array or similar

function rowClassName({ index }) {
  const rowData = list[index];
  // Return a height
}

function MyTable(tableProps) {
  return <Table {...tableProps} rowClassName={rowClassName} />;
}

The same approach could be applied with a class component:

const MyTable extends Component {
  render() {
    return <Table {...tableProps} rowClassName={this._rowClassName} />;
  }
  _rowClassName = ({ index }) => {
    const rowData = this.props.list[index];
    // Return a height
  }
}

[Janpot]

No, it wouldn’t, because RV isn’t guaranteed to start on an “even” or “odd” row given how it works. This would cause the background colors to shift as you scrolled down.

I stand corrected