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

React Dom Translation #126

Merged
merged 17 commits into from
Mar 1, 2019
Merged

React Dom Translation #126

merged 17 commits into from
Mar 1, 2019

Conversation

akhudiakov97
Copy link
Contributor

No description provided.

@akhudiakov97 akhudiakov97 mentioned this pull request Feb 13, 2019
88 tasks
@another-guy another-guy added the needs review A pull request ready to be reviewed label Feb 13, 2019
@lex111
Copy link
Member

lex111 commented Feb 13, 2019

@akhudiakov97 приветствую, пожалуйста, ознакомтесь с руководством, поскольку сейчас много ошибок, которые можно исправить, ознакомившись с этим файлом.

@lex111 lex111 added work in progress Author opened the pull-request but it's still being worked on and removed needs review A pull request ready to be reviewed labels Feb 13, 2019
@akhudiakov97
Copy link
Contributor Author

@lex111 Понял, завтра исправлю, сейчас только увидел

Здравствуйте, вот изменения, не можете подсказать, как правильно перевести hydrate?
Copy link
Member

@lex111 lex111 left a comment

Choose a reason for hiding this comment

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

Посмотрите ещё раз руководство, есть те же ошибки.

Насчёт перевода hydrate можно посмотреть здесь.

content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
Вот, исправил все замечания, и заметил несколько новых ошибок, готов исправить ещё, если будет необходимость )
Вот, исправил все замечания, и заметил несколько новых ошибок, готов исправить ещё, если будет необходимость )
@akhudiakov97
Copy link
Contributor Author

Добрый вечер @lex111, всё хорошо, или нужно что то поправить ?)

Copy link
Member

@lex111 lex111 left a comment

Choose a reason for hiding this comment

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

@akhudiakov97 спасибо, но перевод по-прежнему нуждается в корректировке, я предложил некоторые изменения, но это ещё не всё. Над последними абзацами нужно ещё работать.
Совет: не злоупотребляйте машинным переводом, его можно использовать в качестве основы, но не использовать его без правок, по крайней мере, пока не будет он восприниматься "по-русски". Так что попробуйте прочитать ещё раз перевод, лучше даже вслух, спросите про себя: "Действительно, я понимаю это? Будет ли это понятно другим, менее опытным?". Посмотрите другие источники, или переводы (если есть, но только для лучшего собственного понимания). И, конечно, не стесняйтесь отходить от буквы оригинала, пишите по своему, главное, чтобы смысл был передан верно и понятно (в этом и заключается цель, и сложность перевода). Да, это непросто, но попробуйте ещё раз :)

content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved

> Note
>
> We don't support older browsers that don't support ES5 methods, but you may find that your apps do work in older browsers if polyfills such as [es5-shim and es5-sham](https://github.com/es-shims/es5-shim) are included in the page. You're on your own if you choose to take this path.
> Мы не поддерживаем более старые браузеры, которые не поддерживают ES5 методы, но вы можете обнаружить, что ваши программы работают в более старых браузерах если полифиллы вроде [es5-shim and es5-sham](https://github.com/es-shims/es5-shim) уже присутствуют на странице. Вы будете сами по себе, если решите пойти этой дорогой.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
> Мы не поддерживаем более старые браузеры, которые не поддерживают ES5 методы, но вы можете обнаружить, что ваши программы работают в более старых браузерах если полифиллы вроде [es5-shim and es5-sham](https://github.com/es-shims/es5-shim) уже присутствуют на странице. Вы будете сами по себе, если решите пойти этой дорогой.
> Мы не поддерживаем старые браузеры, в которых отсутствуют ES5-методы, но чтобы обеспечить работу ваших приложений в старых браузерах вы можете подключить шимы [es5-shim и es5-sham](https://github.com/es-shims/es5-shim). Но учтите, мы не сможем вам помочь, если вы предпочтёте сделать это.

Хотя всё равно деревянно (You're on your own if you choose to take this path.), поэтому прошу помощи @another-guy

content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
@akhudiakov97
Copy link
Contributor Author

Окей, понял, исправлю )

Надеюсь стало лучше
@netlify
Copy link

netlify bot commented Feb 20, 2019

Deploy preview for ru-reactjs ready!

Built with commit f8e5b2c

https://deploy-preview-126--ru-reactjs.netlify.com

@akhudiakov97
Copy link
Contributor Author

@lex111 Добрый вечер, как вам последняя версия?
Прошу прощения за беспокойство )

@another-guy
Copy link
Collaborator

@lex111 пройдись, пожалуйста, по исправлениям, я не хочу встрявать в ваш процесс. Я так понял, что suggested changes не были приняты через GitHub напрямую, поэтому мне сложно понять, в каком состоянии сейчас документ.

Как решите вопросы, я могу сделать вычитку целиком -- просто переназначь PR на меня. Пойдёт такой план?

@akhudiakov97 спасибо за работу над переводом! 👍

@akhudiakov97
Copy link
Contributor Author

И вам спасибо @another-guy , я подумаю, может ещё как то улучшу завтра с утра.

Copy link
Member

@lex111 lex111 left a comment

Choose a reason for hiding this comment

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

Увы, по-прежнему нужна работа над переводом, причём не все из моих предложенных правок были включены. Поэтому, пожалуйста, @akhudiakov97, примите их в изначальном виде, просто нажав на Commit suggestion.

content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
content/docs/reference-react-dom.md Outdated Show resolved Hide resolved
@lex111 lex111 assigned another-guy and unassigned lex111 Feb 28, 2019
lex111 and others added 5 commits February 28, 2019 12:21
Co-Authored-By: akhudiakov97 <akhudiakov97@gmail.com>
Co-Authored-By: akhudiakov97 <akhudiakov97@gmail.com>
Co-Authored-By: akhudiakov97 <akhudiakov97@gmail.com>
Co-Authored-By: akhudiakov97 <akhudiakov97@gmail.com>
Co-Authored-By: akhudiakov97 <akhudiakov97@gmail.com>
lex111 and others added 4 commits February 28, 2019 12:22
Co-Authored-By: akhudiakov97 <akhudiakov97@gmail.com>
Co-Authored-By: akhudiakov97 <akhudiakov97@gmail.com>
Co-Authored-By: akhudiakov97 <akhudiakov97@gmail.com>
Co-Authored-By: akhudiakov97 <akhudiakov97@gmail.com>
@lex111
Copy link
Member

lex111 commented Feb 28, 2019

@another-guy теперь можешь ты глянуть, внести свои изменения, а потом слить?

Copy link
Collaborator

@another-guy another-guy left a comment

Choose a reason for hiding this comment

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

Внёс ряд изменений.

@another-guy another-guy added needs proofread A read-through is still needed to release the document and removed work in progress Author opened the pull-request but it's still being worked on labels Mar 1, 2019
@another-guy another-guy merged commit 66fe693 into reactjs:master Mar 1, 2019
@another-guy another-guy removed their assignment Mar 2, 2019
@akhudiakov97 akhudiakov97 deleted the patch-1 branch March 3, 2019 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs proofread A read-through is still needed to release the document
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants