Document React.addons.batchedUpdates

[glenjamin]

I didn't realise this was in the addons bundle, would be good to get it added to the docs.

http://facebook.github.io/react/docs/addons.html
https://github.com/facebook/react/blob/master/src/browser/ReactWithAddons.js#L40

Would be good to also include an example of using https://github.com/facebook/react/blob/master/src/addons/ReactRAFBatchingStrategy.js or something similar.

[sophiebits]

rAF batching isn't supported and won't go in the docs, but batchedUpdates should.

[glenjamin]

is rAF batching discouraged, or just not suitable for shipping with core?

Would it be reasonable to link to an up-to-date npm project on the batchedUpdates page?

[sophiebits]

We don't use it at Facebook and it's not clear that it solves real problems, so we don't want to maintain it or have to fix bugs related to it. React.addons.batchedUpdates should give enough control over batching.

[zpao]

There are also known issues with rAF batching (mostly around controlled components).

[kassens]

Should we deprecate and eventually remove the RAF batching?

[sophiebits]

It's never been supported in any fashion.

[sophiebits]

(i.e., it's already deprecated?)

[chicoxyzzy]

shouldn't this file be deleted in that case?
https://github.com/facebook/react/blob/master/src/addons/ReactRAFBatchingStrategy.js

[sophiebits]

That's not exposed as a public API in any of our builds, but sure, we can delete it – PR incoming.

[jimfb]

Killed in #6016

[sophiebits]

We should document ReactDOM.unstable_batchedUpdates.

[jimfb]

@spicyj Since when did we start documenting unstable_ features? Haven't we always left them intentionally undocumented? We also have unstable_renderSubtree and unstable_handleError and others, but I thought the decision was to not document them until they become stable. We document them as part of the release process.

It was a different situation when it was an addon, the issue made sense at that time.

[sophiebits]

Sure.