Skip to content

Conversation

@dmtrKovalenko
Copy link
Contributor

@dmtrKovalenko dmtrKovalenko commented Feb 13, 2019

  1. Перевел polled как делегированные - не знаю насколько это правильно
  2. Стоит ли переводить название событий (мне кажется это только будет сбивать с толку)

@tesseralis tesseralis 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 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 15, 2019
@netlify
Copy link

netlify bot commented Feb 21, 2019

Deploy preview for ru-reactjs ready!

Built with commit c34d134

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

@another-guy
Copy link
Collaborator

another-guy commented Feb 22, 2019

@dmtrKovalenko @lex111

> 1. Перевел polled как делегированные - не знаю насколько это правильно

Как вам вариант "опрашиваемые"? Это наиболее близко к страдательному/пассивному от poll -- опрашивать. Мне кажется, так чаще переводится.

Разница между polling/delegation только в том, "клиент" или "сервер" являются активной стороной.

Мне нужно разуть 👀 и купить 👓 .

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.

Хорошо бы немного отшлифовать. В целом, всё хорошо! Спасибо @dmtrKovalenko! 👍 👍 👍

@lex111
Copy link
Member

lex111 commented Feb 22, 2019

Как вам вариант "опрашиваемые"? Это наиболее близко к страдательному/пассивному от poll -- опрашивать. Мне кажется, так чаще переводится.

Разница между polling/delegation только в том, "клиент" или "сервер" являются активной стороной.

Что-то совсем всё запуталось. Не polling, а pooling, тут вообще делегирование ни к месту. Речь идёт про пул. Соответственно is pooled просто означает, что событие помещается в пул, либо находится в нём

@another-guy
Copy link
Collaborator

another-guy commented Feb 22, 2019

@lex111 всё так! Я не знаю, как я столько раз неправильно читал оригинал. 🤦‍♂️ 🤦‍♂️ 🤦‍♂️

@dmtrKovalenko но всё равно надо подобрать перевод для pooled везде.

@dmtrKovalenko
Copy link
Contributor Author

@another-guy так какой перевод?)

@another-guy
Copy link
Collaborator

@dmtrKovalenko, я сделал commit suggestions. Принимай, и можем смёрджить. 🎉

@dmtrKovalenko
Copy link
Contributor Author

dmtrKovalenko commented Feb 24, 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.

Огромное спасибо за вклад, @dmtrKovalenko! 🎉 🤝

@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 Feb 25, 2019
@another-guy another-guy merged commit 61b78b8 into reactjs:master Feb 25, 2019
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.

5 participants