Skip to content

Commit 8968fd6

Browse files
committed
docs(migration): fix key value + adding close tag for CSSTransition when needed
1 parent 91f9b86 commit 8968fd6

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

Migration.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ class TodoList extends React.Component {
6363

6464
render() {
6565
const items = this.state.items.map((item, i) => (
66-
<div key={item} onClick={() => this.handleRemove(i)}>
66+
<div key={i} onClick={() => this.handleRemove(i)}>
6767
{item}
6868
</div>
6969
));
@@ -88,7 +88,7 @@ The most straightforward way to migrate is to use `<TransitionGroup>` instead of
8888
```diff
8989
render() {
9090
const items = this.state.items.map((item, i) => (
91-
<div key={item} onClick={() => this.handleRemove(i)}>
91+
<div key={i} onClick={() => this.handleRemove(i)}>
9292
{item}
9393
</div>
9494
));
@@ -141,14 +141,14 @@ Now we add the `<CSSTransition>` component:
141141
render() {
142142
const items = this.state.items.map((item, i) => (
143143
+ <CSSTransition
144-
+ key={item}
144+
+ key={i}
145145
+ classNames="example"
146146
+ timeout={{ enter: 500, exit: 300 }}
147147
+ >
148148
<div onClick={() => this.handleRemove(i)}>
149149
{item}
150150
</div>
151-
+ <CSSTransition>
151+
+ </CSSTransition>
152152
));
153153

154154
return (
@@ -184,15 +184,15 @@ We can then use it like:
184184
render() {
185185
const items = this.state.items.map((item, i) => (
186186
- <CSSTransition
187-
- key={item}
187+
- key={i}
188188
- classNames="example"
189189
- timeout={{ enter: 500, exit: 300 }}
190190
- >
191191
+ <FadeTransition>
192192
<div onClick={() => this.handleRemove(i)}>
193193
{item}
194194
</div>
195-
- <CSSTransition>
195+
- </CSSTransition>
196196
+ </FadeTransition>
197197
));
198198

0 commit comments

Comments
 (0)