Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Finished translation of Suspense page #556

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

GeorgeShvab
Copy link
Contributor

Apologies for the long time this translation has taken.

Translated "fallback" as "запасний варіант" according to the glossary. I considered other variants but couldn't find a more suitable option.

Copy link

vercel bot commented Jul 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
uk-legacy-reactjs-org ⬜️ Ignored (Inspect) Visit Preview Oct 17, 2024 0:38am

Copy link

github-actions bot commented Jul 28, 2024

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! 🙌

@alinkedd
Copy link
Collaborator

Thanks for the PR!

I'll do a review a little bit later, as there are already another two big PRs submitted earlier which are waiting for the review.

Copy link
Collaborator

@alinkedd alinkedd left a comment

Choose a reason for hiding this comment

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

Перша частина перевірки

src/content/reference/react/Suspense.md Outdated Show resolved Hide resolved
Comment on lines 27 to 28
* `children`: Реальний UI, який ви хочете відрендерити. Якщо `children` затримується під час рендерингу, межа Suspense перемкнеться на рендер `fallback`.
* `fallback`: Альтернативний UI, який рендериться замість справжнього UI, якщо той ще не завершив завантаження. Будь-який валідний React-вузол приймається, хоча на практиці, запасний варіант є легким заповнювачем, таким як спінер чи скелетон. Suspense автоматично перемкнеться на `fallback`, коли `children` затримується, і назад на `children`, коли дані будуть готові. Якщо `fallback` затримується під час рендеру, найближча батьківська межа Suspense буде активована.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Треба або в двох використовувати "реальний UI", або "справжній UI", щоб був однаковий переклад для "actual UI". Також я б переклала як "бажаний UI" або "очікуваний UI", щоб краще передати суть.

src/content/reference/react/Suspense.md Outdated Show resolved Hide resolved
src/content/reference/react/Suspense.md Show resolved Hide resolved
src/content/reference/react/Suspense.md Outdated Show resolved Hide resolved
* `children`: The actual UI you intend to render. If `children` suspends while rendering, the Suspense boundary will switch to rendering `fallback`.
* `fallback`: An alternate UI to render in place of the actual UI if it has not finished loading. Any valid React node is accepted, though in practice, a fallback is a lightweight placeholder view, such as a loading spinner or skeleton. Suspense will automatically switch to `fallback` when `children` suspends, and back to `children` when the data is ready. If `fallback` suspends while rendering, it will activate the closest parent Suspense boundary.
#### Пропси {/*props*/}
* `children`: Реальний UI, який ви хочете відрендерити. Якщо `children` затримується під час рендерингу, межа Suspense перемкнеться на рендер `fallback`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* `children`: Реальний UI, який ви хочете відрендерити. Якщо `children` затримується під час рендерингу, межа Suspense перемкнеться на рендер `fallback`.
* `children`: Реальний UI, який ви хочете відрендерити. Якщо `children` затримується (suspends) під час рендерингу, межа (boundary) Suspense перемкнеться на рендер `fallback`.

Я погоджуюсь тут із boundary як межа і suspend як затримувати, просто додамо на першому входженні оригінал для розуміння

p.s. чому suspend не призупиняти? є якийсь окремий підтекст чи пояснення?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Підтексту немає, можна замінити на призупиняти за необхідності

src/content/reference/react/Suspense.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@alinkedd alinkedd left a comment

Choose a reason for hiding this comment

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

Друга частина перевірки: усі ці та попередні зауваження мають бути виправлені у всьому PR.

Третя частина перевірки буде після злиття #564 і виправлень першої і другої частини перевірки цього PR - інакше у мене накладається складність коректури одного і того самого

src/content/reference/react/Suspense.md Outdated Show resolved Hide resolved
@@ -584,9 +584,9 @@ function Details({ artistId }) {

---

### Revealing nested content as it loads {/*revealing-nested-content-as-it-loads*/}
### Відображення вкладеного вмісту по мірі його завантаження {/*revealing-nested-content-as-it-loads*/}
Copy link
Collaborator

Choose a reason for hiding this comment

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

тут і далі - виправити вживання прийменника по
https://onlinecorrector.com.ua/uk/%D0%BF%D0%BE/

src/content/reference/react/Suspense.md Outdated Show resolved Hide resolved
src/content/reference/react/Suspense.md Outdated Show resolved Hide resolved
src/content/reference/react/Suspense.md Outdated Show resolved Hide resolved
src/content/reference/react/Suspense.md Outdated Show resolved Hide resolved
src/content/reference/react/Suspense.md Outdated Show resolved Hide resolved
@GeorgeShvab
Copy link
Contributor Author

Дякую за рев'ю.

  • Виправив вказані помилки
  • Ще кілька разів перечитав переклад, зробив інші виправлення
  • Додав fallback як запасний варіант у TRANSLATION.md

@alinkedd
Copy link
Collaborator

@GeorgeShvab можеш ребейзнути?

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