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

Translate "Forms" #174

Merged
merged 31 commits into from
Mar 1, 2019
Merged

Translate "Forms" #174

merged 31 commits into from
Mar 1, 2019

Conversation

thechch
Copy link
Contributor

@thechch thechch commented Feb 20, 2019

No description provided.

Copy link
Collaborator

@Heegiiny Heegiiny left a comment

Choose a reason for hiding this comment

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

Внутри слова используется дефис, а не тире, не нужны запятые после В React. Плюс некоторые предложения по стилю.

content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
@Heegiiny Heegiiny added the needs review A pull request ready to be reviewed label Feb 20, 2019
@Heegiiny
Copy link
Collaborator

По поводу единственный источник правды. Источник достоверных данных лучше звучит на мой взгляд, но правды гораздо чаще встречается на хабре.

@thechch
Copy link
Contributor Author

thechch commented Feb 21, 2019

@Heegiiny Огромное спасибо за ревью!

По поводу единственный источник правды. Источник достоверных данных лучше звучит на мой взгляд, но правды гораздо чаще встречается на хабре.

В таком случае, стоит выбрать то, что чаще встречается в переводе на этом сайте. Мне не очень нравится "источник правды" потому что это больше похоже на отсылку к некоему фентези 😺

Еще вопрос. DOM-элемент, React-компонент но React-приложение или все же React приложение?

@netlify
Copy link

netlify bot commented Feb 21, 2019

Deploy preview for ru-reactjs ready!

Built with commit 3b8a461

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

@Heegiiny
Copy link
Collaborator

Я думаю, это общее правило:
DOM-элемент, элемент DOM
React-компонент, компонент React
React-приложение, приложение на React

То есть в первом случае это как бы приставка к слову, поэтому через дефис, а во втором отдельное слово, и его стоит подбирать чтобы хорошо звучала транслитерация.

@Heegiiny
Copy link
Collaborator

По поводу источника истины тут была дискуссия, @lex111 избавился от словосочетания, заменив его на просто данные

Since an uncontrolled component keeps the source of truth in the DOM

Такие компоненты хранят данные в DOM

Встречается ещё здесь, но эту страницу ещё не перевели.

@tesseralis tesseralis mentioned this pull request Feb 21, 2019
88 tasks
@thechch
Copy link
Contributor Author

thechch commented Feb 21, 2019

Окай, обновлю вечером 👍

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.

Предложения по улучшению перевода. Можете их принять непосредственно с гитхаба, поэтому для того, чтобы помощь участникам мы предлагаем изменения (suggested change), а не просто пишем их. Таким образом вы легко можете применить их.

content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@Heegiiny Heegiiny left a comment

Choose a reason for hiding this comment

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

Ещё несколько предложений по изменению стиля.

content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
content/docs/forms.md Outdated Show resolved Hide resolved
@another-guy another-guy self-assigned this Feb 22, 2019
lex111 and others added 10 commits February 23, 2019 21:35
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
@thechch thechch changed the title Translate "Forms" Translate "Forms": rephrasing Feb 23, 2019
Heegiiny and others added 10 commits February 23, 2019 23:03
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
Co-Authored-By: thechch <thech.ch@gmail.com>
@thechch thechch changed the title Translate "Forms": rephrasing Translate "Forms" Feb 25, 2019
@another-guy another-guy removed their assignment Mar 1, 2019
@another-guy another-guy added needs proofread A read-through is still needed to release the document and removed needs review A pull request ready to be reviewed labels Mar 1, 2019
@another-guy another-guy merged commit fedaf42 into reactjs:master Mar 1, 2019
@another-guy
Copy link
Collaborator

Спасибо @thechch ! Очень хороший результат! 👍

@thechch
Copy link
Contributor Author

thechch commented Mar 2, 2019

Спасбио @Heegiiny @another-guy @lex111 за ревью и правки!

@another-guy а что про источник правды то решили?

@another-guy
Copy link
Collaborator

@thechch стоит открыть ишью и решить?

Лично я с этим понятием сталкиваюсь постоянно, потому что только английские доки и читаю, а там это сплошь и рядом. Моё мнение, что нет ничего плохого в том, чтобы познакомить русскоязычного читателя с термином, даже если это и калька. Просто это очень полезная концепция. И не только в контексте React.

Copy link
Collaborator

@Heegiiny Heegiiny left a comment

Choose a reason for hiding this comment

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

Отлично, сорри, занят, забыл подтвердить, к тому же ждал, что скажут @lex111 и @another-guy

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.

4 participants