-
Notifications
You must be signed in to change notification settings - Fork 385
Translate "State and Lifecycle" (state-and-lifecycle.md) #42
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Спасибо! Начал смотреть перевод, но при беглом осмотре увидел много отклонений от руководства по переводу, пожалуйста, ознакомтесь с ним, и скорректируйте перевод, позже будет легче снова делать ревью перевода.
@lex111 спасибо, я раньше не переводил. Насколько мешает (довольно) вольная интерпретация перевода? Например, я иногда менял местами части предложения чтобы оживить и точнее передать смысл - это допустимо? Или вы имеете в виду только употребление терминов? |
Это в основном, да. |
@agoldis там ещё проблемы со знаками пунктации, в принципе это легко поправить (заменить дефисы на тире). |
Надо еще заголовок поменять в |
О! Может добавить соответствующий пункт-чекбокс в шаблон пулреквеста? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Хорошее начало, первые замечания:
- мелкие отличия от терминологии
- кавычки «»
- местами, нагромождения существительных, которые читаются сложнее оригинального текста
Давайте поправим сначала эти проблемы. Я не читал до конца, т.к. в целом проблемы похожи, но когда поправите, почитаю ещё раз заново. Спасибо!
@gaearon @another-guy thanks guys for your feedback, please check out the updated version |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- можно перевести текст в блоках кода
Мне кажется, лучше переводить, мы переводом единое руководство, где написано, что необязательно, но можно переводить, и поскольку на главной странице уже переведены надписи в примерах, то для согласованности предлагаю придерживаться этого варианта. |
Хорошо, было бы здорово прояснить в правилах этот ньюанс :) |
Сделаем в #61 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
В целом уже хорошо. Несколько замечаний, потом можно мержить. Большое спасибо;
@gaearon спасибо за комментарии! ознакомься плз с новой версией |
content/docs/state-and-lifecycle.md
Outdated
|
||
4) Every second the browser calls the `tick()` method. Inside it, the `Clock` component schedules a UI update by calling `setState()` with an object containing the current time. Thanks to the `setState()` call, React knows the state has changed, and calls the `render()` method again to learn what should be on the screen. This time, `this.state.date` in the `render()` method will be different, and so the render output will include the updated time. React updates the DOM accordingly. | ||
4) Таймер вызывает метод `tick()` ежесекундно. Часть метода — вызов `setState()` с объектом текущего времени — таким образом компонент `Clock` планирует обновление UI. Благодаря вызову `setState()` React знает, что состояние изменилось, и снова вызывает метод `render()`, чтобы узнать, что должно отображаться на экране. На этот раз `this.state.date` в методе `render()` будет другим, рендер компонента будет включать обновлённое время и React обновит DOM соответственно. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Часть метода — вызов
setState()
с объектом текущего времени — таким образом компонентClock
планирует обновление UI
Непонятная структура
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
рендер компонента будет включать обновлённое время и React обновит
Нужна запятая, сложное предложение
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gaearon посмотри пожалуйста новый вариант этого блока, я решил переписать после твоего отзыва. Спасибо!
Отлично. |
* upstream/master: (108 commits) Update nav Translation List and keys (reactjs#64) Translate "State and Lifecycle" (state-and-lifecycle.md) (reactjs#42) Add the term fallback Update getting-started.md Update getting-started.md Update TRANSLATION.md Update getting-started.md Update getting-started.md Update TRANSLATION.md Терминология: уборка Исправить Условный рендеринг в нав панели Пропущенное исправление в conditional-rendering Терминология: code splitting Терминология: helper Терминология child [component] Упорядочить глоссарий по алфавиту Больше исправлений по замечаниям касательно conditional-rendering Исправил PR-замечания про conditional-rendering.md Translate "Conditional Rendering" into Russian ...
Правда там локальное должно быть внутренним. |
Спасибо @lex111 за https://github.com/js-ru/react-ru/blob/master/content/docs/state-and-lifecycle.md - я использовал перевод как базу и референс, но изменил стилистику и переписал некоторые фразы, руководствуясь https://github.com/reactjs/ru.reactjs.org/blob/master/TRANSLATION.md