tic-toe #1

OlgaTitarenko · 2019-02-21T17:56:47Z

No description provided.

ivadimko

В целом хорошо
Оставил комментарии, что можно было бы улучшить

ivadimko · 2019-02-21T18:12:09Z

src/App.js

+  }
+
+  handleClick(i) {
+    const square = this.state.squares.slice();


slice же только для копирования массива?
более обыденным выглядело бы

const square = [...this.state.squares]

ivadimko · 2019-02-21T18:19:44Z

src/App.js

+          {' '}
+        </div>
+
+        <div className="board-row">


было бы хорошо автоматизировать рендер ячеек
можно сделать позиционирование стилями и избавиться от промежуточных board-row

this.state.squares.map((square, i) => this.renderSquare(i, winLine))

tic-toe

ef9fb47

ivadimko approved these changes Feb 21, 2019

View reviewed changes

tic-toe final changes

b339c28

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

tic-toe #1

tic-toe #1

OlgaTitarenko commented Feb 21, 2019

ivadimko left a comment

ivadimko Feb 21, 2019

ivadimko Feb 21, 2019

tic-toe #1

Are you sure you want to change the base?

tic-toe #1

Conversation

OlgaTitarenko commented Feb 21, 2019

ivadimko left a comment

Choose a reason for hiding this comment

ivadimko Feb 21, 2019

Choose a reason for hiding this comment

ivadimko Feb 21, 2019

Choose a reason for hiding this comment