Add support to Table to allow components that extend Column

[CptLemming]

This change allows the Table component to accept children that directly extend the Column component.

This allows me to create many Column types I can use directly in a Table, making code reuse simpler.

Creating custom columns

// Changing the default cellDataGetter to use lodash's `get` function
const cellDataGetter = ({ dataKey, rowData }) => {
  return get(rowData, dataKey);
};

export class GetColumn extends Column {
  static defaultProps = {
    ...Column.defaultProps,
    cellDataGetter: cellDataGetter,
  };
}

// Adding a column to render links using `react-router`
const cellLinkRenderer = ({ cellData, rowData }) => {
  if ('to' in rowData) {
    let linkUrl = rowData.to;
    if (typeof rowData.to === 'function') {
      linkUrl = rowData.to({ rowData, cellData });
    }
    return (<Link to={ linkUrl }>{ cellData }</Link>);
  }
  return cellData;
};

export class LinkColumn extends Column {
  static defaultProps = {
    ...Column.defaultProps,
    cellRenderer: cellLinkRenderer,
  };
}

Using custom columns

const list = [{ node: { name: 'foo' }, link: '/bar' }];

<Table
  height={100}
  width={100}
  rowCount={list.length}
  rowGetter={({ index }) => list[index]}
>
  <GetColumn
    label="Name"
    dataKey="node.name"
    width={50}
  />
  <LinkColumn
    label="More info"
    dataKey="link"
    width={50}
  />
</Table>

[bvaughn]

This (custom columns) was already possible, using a different syntax, but I think this is a reasonable simplifying change. 😄 Thanks.

[bvaughn]

This feature has been released in version 9.10.0. Thank you for contributing!

[genomics-geek]

Is there a way to do this with composition instead of inheritance? Seems like Table component is very strict on only accepting instance of Column

[bvaughn]

Not currently.
https://github.com/bvaughn/react-virtualized/blob/master/source/Table/Table.js#L30-L38

[genomics-geek]

@bvaughn - Would you be against changing this to PropTypes.node ? Is there something preventing making this change?

[bvaughn]

It just doesn't feel super compelling to me. I guess I'm not really against the idea but I don't intend to make the change b'c I don't see it as a pressing need and I'm super strapped for time.

[genomics-geek]

that's fair - It's already an awesome package :).

I will submit a PR for this, you can decide later if this makes sense to include it or not in a future release. Thanks for this package!