Edit the description of componentDidMount() to make clear that the function is called only once during the lifecycle of the component, after only the initial render() call. The description is now similar to that for componentWillMount(). The existing wording does not make it clear that the function is called only after the initial render. (Granted, it is possible to infer this from descriptions of the component lifecycle found elsewhere in the documentation, but I think explicit is better than implicit in this case).
The reason I chose to submit this is because it was initially unclear from the docs that the transition group needed to already be mounted in order for it to work properly.
Added docs for `React.isValidClass` and `React.isValidComponent`.
Still missing top level api:
* `constructAndRenderComponent`
* `constructAndRenderComponentById`
* `withContext`
When I first read these docs, it was not immediately clear to me that I could
use `React.addons.CSSTransitionGroup` to animate a single item coming into view,
or an item replacing an item already there. This was partly due to the example
which rendered a list of items.
This PR adds a blurb about being able to use
`React.addons.CSSTransitionGroup` with one or zero items, provides a code
example, and adds a note blockquote that a `key` attribute must always be
provided on each child of `React.addons.CSSTransitionGroup`. The latter point
was not immediately obvious from the original `TodoList` code example, since it
renders a list which would normally require `key` attributes anyway.
Test Plan:
- Refreshed `http://localhost:4000/react/docs/animation.html`, saw that the
docs additions rendered correctly.
- Example code not tested (it was extracted from working code).
Mark Hintz
Simon Welsh
Rick Beerendonk
jon madison
Nick Merwin
Naman Goel
Paul O’Shannessy
Mike D Pilsbury
Julen Ruiz Aizpuru
fisherwebdev
Michael Randers-Pehrson
IvanVergiliev
Chris Sciolla
Dmitrii Abramov
Michael Chan
Charles Marsh
Andre Z Sanchez
Nathan White
Adam Mark
Sam Selikoff
Jan Hancic
Jan Hancic
Ben Alpert
Julien Bordellier
Jim OBrien
cutbko
Sébastien Lorber
David Hu
WickyNilliams
Cheng Lou