Skip to content

Conversation

westanvv
Copy link
Contributor

Added translation for Components and Props page.

@vldmrkl vldmrkl self-requested a review February 13, 2019 22:14
@mihilbabin
Copy link
Contributor

mihilbabin commented Feb 14, 2019

Можливо, є сенс словосполучення витягнути компонент замінити на виокремити компонент. It's up to you.

Copy link
Member

@vldmrkl vldmrkl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Гарний початок! Кілька дрібних виправлень, а загалом дуже добре.

4. React DOM efficiently updates the DOM to match `<h1>Hello, Sara</h1>`.
1. Ми викликаємо `ReactDOM.render()` з елементом `<Welcome name="Василина" />`.
2. React викликає компонент `Welcome` з пропсом `{name: 'Василина'}`.
3. `Welcome` компонент повертає елемент `<h1>Привіт, Василина</h1>`.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Повинен використовуватися кличний відмінок: Привіт, Василино

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я б не змінював, адже name передбачає ім'я в називному відмінку. А відмінювання - це вже логіка відображення. Що вирішуємо? Змінити?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@westanvv гарний аргумент. Залишаємо тоді у називному відмінку.

1. Ми викликаємо `ReactDOM.render()` з елементом `<Welcome name="Василина" />`.
2. React викликає компонент `Welcome` з пропсом `{name: 'Василина'}`.
3. `Welcome` компонент повертає елемент `<h1>Привіт, Василина</h1>`.
4. React DOM ефективно оновлює DOM для отримання `<h1>Привіт, Василина</h1>`.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Повинен використовуватися кличний відмінок: Привіт, Василино

@vldmrkl vldmrkl merged commit 2bd8a1d into reactjs:master Feb 19, 2019
@westanvv westanvv deleted the сomponents-and-props branch April 13, 2019 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants