Skip to content

Conversation

@vadym-oliinyk
Copy link
Contributor

No description provided.

Перенесіть компонент `Profile` у окремий файл `Profile.js`, а потім змініть компонент `App` так, щоб він рендерив `<Profile />` та `<Gallery />` один за іншим.

Move the `Profile` component to its own `Profile.js`, and then change the `App` component to render both `<Profile />` and `<Gallery />` one after another.
Ви можете використовувати як дефолтний, так і іменований експорт для `Profile`, проте переконайтеся, що використовуєте відповідний синтаксис імпорту як в `App.js`, так і в `Gallery.js`! Ви можете скористатися таблицею з розділу про поглиблений аналіз вище:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Тут "deep dive" переклав як "поглиблений аналіз". Але не впевнений, що це доречно для цього глобального компоненту

* How to split components into multiple files
* Що таке файл кореневого компонента
* Як імпортувати та експортувати компонент
* Коли використовувати дефолтні та іменовані імпорти та експорти
Copy link
Contributor Author

@vadym-oliinyk vadym-oliinyk May 11, 2023

Choose a reason for hiding this comment

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

По всьому тексту переклав "default exports" як "дефолтні експорти", а не "експорти за замовчуванням". Чи це окей?

@bkatsevych
Copy link
Collaborator

Гарна робота!

@github-actions
Copy link

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@bkatsevych bkatsevych merged commit a4948ae into reactjs:main May 25, 2023
@vadym-oliinyk
Copy link
Contributor Author

@bkatsevych дякую! внести ті корективи, що в коментарях вище? чи вони вже resolved бо не актуальні?)

@bkatsevych
Copy link
Collaborator

@vadym-oliinyk Після ревʼю, ваші + мої корективи вступають в силу, перед тим як замержити PR.

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.

2 participants