Support flow type props validation

[ndelangen]

Did you have any plan to support:

export default class Item extends React.Component { // eslint-disable-line
  props: {
    icon: string,
    tooltip?: string,
    onClick?: () => void,
  };

  render() {
    const { icon, tooltip, onClick } = this.props
    return (
      <span className={styles.root} onMouseDown={onClick} title={tooltip}>
        <span className="material-icons">{icon}</span>
      </span>
    )
  }
}

together with

  static propTypes = {
    icon: React.PropTypes.string.isRequired,
    tooltip: React.PropTypes.string,
    onClick: React.PropTypes.func,
  };

[danielduan]

I'm not actually sure how flow works under the hood. If there's a way to get flow type transpiled in babel to static propTypes before we process the docgen stuff, there's definitely possibility.

[danielduan]

Gonna close this one since it seems like react-docgen has support for flow-types in the latest versions:
https://github.com/reactjs/react-docgen

[Mottoweb]

It seems that info addon still does not render props from flow types though.

[danielduan]

@Mottoweb try cleaning out your node_modules and reinstalling. You'll need the latest version of info and the associated babel plugin. It's working in the example app out of the box.

[alexeyMohnatkin]

Should it work with flow types only?

[danielduan]

I'm not sure what you mean by flow types only @alexeyMohnatkin. This should work with both regular prop types and flow types.

[Mottoweb]

@danielduan it worked thanks, but still i cant see the description for the prop. Or maybe its addon-info related?

[danielduan]

@Mottoweb there's been a bug fix merged to 3.3 that fixes the docgen issues when loading from static bundles.