Autobinding instead of binding in the constructor

[nikgraf]

See https://facebook.github.io/react/blog/2015/01/27/react-v0.13.0-beta-1.html#autobinding

Prebinding in the constructor:

class Counter extends React.Component {
  constructor() {
    super();
    this.tick = this.tick.bind(this);
  }
  tick() {
    ...
  }
  ...
}

Syntactical sugar:

class Counter extends React.Component {
  tick = () => {
    ...
  }
  ...
}

[mxstbr]

Oooh I didn't know about that, much nicer! Would love a PR for this.

[jbinto]

FWIW, that syntax (class fields/static properties) isn't an ES6 feature, it is a stage 1 proposal that has support only in Babel.

I don't have a problem with using experimental syntax but I definitely think it's worth documenting anything like this, since it could change on a whim. Docs should make it clear the project relies on experimental language features, which ones, and why.

[mxstbr]

👍 Thanks for the notice, I hope I don't forget to document it!

[jbinto]

I opened #165 to track it.

[mxstbr]

Thanks @jbinto, closing this issue!

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.