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

Translation for State as a Snapshot page #877

Merged
merged 6 commits into from
Aug 8, 2023

Conversation

bofeof
Copy link
Contributor

@bofeof bofeof commented May 10, 2023

Если ваш пулреквест является исправлением бага, а не переводом, то сперва убедитесь, что проблема относится ТОЛЬКО к https://ru.reactjs.org, а не к https://reactjs.org. Если это не так, то пулреквест следует открыть в родительском репозитории.

@bofeof bofeof changed the title Translation for state-as-a-snapshot page Translation for State as a Snapshot page May 11, 2023
Copy link
Collaborator

@titovmx titovmx left a comment

Choose a reason for hiding this comment

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

привет, посмотрел перевод, в целом всё понравилось! Оставил пару небольших замечаний

---

<Intro>

State variables might look like regular JavaScript variables that you can read and write to. However, state behaves more like a snapshot. Setting it does not change the state variable you already have, but instead triggers a re-render.
На первый взгляд, переменные состояния выглядят как обычные JavaScript переменные, с которыми вы можете проводить операции чтения и записи. Однако, состояние больше похоже на снимок, чем на классическую переменную. Установка нового значения переменной состояния не изменяет напрямую текущее состояние, но при этом инициирует повторный рендер.
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
На первый взгляд, переменные состояния выглядят как обычные JavaScript переменные, с которыми вы можете проводить операции чтения и записи. Однако, состояние больше похоже на снимок, чем на классическую переменную. Установка нового значения переменной состояния не изменяет напрямую текущее состояние, но при этом инициирует повторный рендер.
На первый взгляд, переменные состояния выглядят как обычные JavaScript-переменные, с которыми вы можете проводить операции чтения и записи. Однако, состояние больше похоже на снимок, чем на классическую переменную. Установка нового значения переменной состояния не изменяет напрямую текущее состояние, но при этом инициирует повторный рендер.

"JavaScript" и "переменная" образуют в данном случае сложное существительное, образованное без соединительной гласной, поэтому следует ставить дефис.

* When and how state updates
* Why state does not update immediately after you set it
* How event handlers access a "snapshot" of the state
* Как установка состояния запускает повторный рендер;
Copy link
Collaborator

Choose a reason for hiding this comment

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

в других переводах, которые я здесь видел, для списков копируется пунктуация из оригинала. В YouWillLearn без какого-либо знака в конце, для маркированных и нумерованных списков в тексте - точка в конце.
Думаю, стоит придерживаться одного стиля или поднять эту тему в топике #851

@bofeof
Copy link
Contributor Author

bofeof commented Jun 21, 2023

@titovmx привет! спасибо большое за обратную связь) замечания справедливые, исправила)

@gcor
Copy link
Collaborator

gcor commented Jul 5, 2023

@bofeof пожалуйста решите конфликт в ветке

«This branch has conflicts that must be resolved»

Copy link
Collaborator

@titovmx titovmx left a comment

Choose a reason for hiding this comment

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

за исключением микрокоммента всё супер 👍

}}>
Change to Stop
Изменить на Стоп
</button>
<h1 style={{color: 'darkgreen'}}>
Walk
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
Walk
Можно идти

один хэдер в примерах пропустили :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

готово!

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

@titovmx titovmx merged commit 2d74b12 into reactjs:main Aug 8, 2023
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.

3 participants