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

Profiler translation #388

Merged
merged 24 commits into from
Sep 4, 2019
Merged

Conversation

IgorLesnevskiy
Copy link
Contributor

@IgorLesnevskiy IgorLesnevskiy commented Aug 24, 2019

Closes #382

  1. Название компонента Profiler оставил на английском как есть, однако все другие вхождения, которые не указывают на название компонента, русифицировал, т.к. в русском языке слово "профилирование" чувствует себя нормально (статья на Википедии)

  2. Согласно гайдам, в заголовке статьи должно быть только одно слово с заглавной буквы. В моем случае это аббревиатура API. Изначально получилось "API профайлера", но я расширил до "API для работы с Profiler". Корректно ли оставить такое название? Возможно стоит поменять на "API для работы с компонентом Profiler" или "API для компонента Profiler".

  3. Встречается такой термин как Set of interactions. Перевел как "Set-набор взаимодействий", но звучит немного неуклюже. Есть предложения?

  4. Я не совсем понимаю как правильно использовать единственное и множественно число для слова "проп":

  • два обязательных пропса или два обязательных пропа
  • каждый из пропсов или каждый из пропов
  • Он принимает два пропса: или Он принимает два пропа:
  1. API для их отслеживания все еще экспериментальное. или API для их отслеживания все еще экспериментально.

  2. Вероятно, стоит добавить Profiler и производные в словарь

@netlify
Copy link

netlify bot commented Aug 24, 2019

Deploy preview for ru-reactjs ready!

Built with commit 2d123fd

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

@ntishkevich ntishkevich added the needs review A pull request ready to be reviewed label Aug 24, 2019
@lex111
Copy link
Member

lex111 commented Aug 26, 2019

Спасибо за перевод, пока не смотрел, но попытался ответить на вопросы:

  1. Ок, под этим имеется в виду название компонента
  2. Корректно, даже лучше так
  3. Хм, пока не вникал в это 🤷‍♂️
  4. два пропа / каждый из пропсов
  5. экспериментальное
  6. Можно сделать в текущем пулреквесте

@IgorLesnevskiy
Copy link
Contributor Author

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

@lex111
Copy link
Member

lex111 commented Aug 26, 2019

@IgorLesnevskiy а я попробую вникнуть в текст, но это не точно :/
@ntishkevich если есть возможность, посмотри, пожалуйста?

@IgorLesnevskiy
Copy link
Contributor Author

@lex111 внес правки, добавил термины (в том числе Set of interactions, но это пока спорно)

Copy link
Collaborator

@ntishkevich ntishkevich left a comment

Choose a reason for hiding this comment

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

Пробежался по переводу, некоторые изменения предложил на обсуждение. Также необходимо перевести секцию Profiler в nav.yml (content/docs/nav.yml)

content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
@IgorLesnevskiy
Copy link
Contributor Author

IgorLesnevskiy commented Aug 30, 2019

@ntishkevich спасибо за исправления. Вроде и правила читал, и вычитал четыре раза, а все равно пропустил :-(

Пробежался по переводу, некоторые изменения предложил на обсуждение. Также необходимо перевести секцию Profiler в nav.yml (content/docs/nav.yml)

Я не стал переводить, т.к. решил, что это имя Компонента. Может тогда туда вынести заголовок статьи?

@IgorLesnevskiy
Copy link
Contributor Author

@lex111 @ntishkevich я обновил тайтл в навигации. я правильно понимаю, что у нас остался только спорный момент с "Set-набор взаимодействий"?

@ntishkevich
Copy link
Collaborator

@lex111 @ntishkevich я обновил тайтл в навигации. я правильно понимаю, что у нас остался только спорный момент с "Set-набор взаимодействий"?

На мой взгляд, да, осталось разобраться с о спорным моментом и все ок.
@lex111 как будет время, можешь посмотреть изменения, также может появится идеи насчет нашего спорного момента?

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

@IgorLesnevskiy
Copy link
Contributor Author

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

да, хорошо, сделаю в ближайшее время

TRANSLATION.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
content/docs/reference-profiler.md Outdated Show resolved Hide resolved
TRANSLATION.md Outdated Show resolved Hide resolved
@IgorLesnevskiy
Copy link
Contributor Author

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

строчки подогнал

@lex111
Copy link
Member

lex111 commented Sep 4, 2019

@ntishkevich если всё ок, то сливай.

Copy link
Collaborator

@ntishkevich ntishkevich left a comment

Choose a reason for hiding this comment

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

@IgorLesnevskiy Спасибо за перевод! 💪

@ntishkevich ntishkevich merged commit 8e82e06 into reactjs:master Sep 4, 2019
@IgorLesnevskiy
Copy link
Contributor Author

@lex111 @ntishkevich спасибо за возможность внести свою вклад в сообщество, парни

@IgorLesnevskiy
Copy link
Contributor Author

IgorLesnevskiy commented Sep 4, 2019

@ntishkevich вопрос - тут точно нужна "ё"? Если не путаю - от вас была правка

Профилирование добавляет накладные расходы, поэтому оно отключёно в продакшен-режиме.

@lex111
Copy link
Member

lex111 commented Sep 4, 2019

@IgorLesnevskiy точно, убрал её 857828a.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs review A pull request ready to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Перевод страницы Profiler
3 participants