-
Couldn't load subscription status.
- Fork 48
Introduce Russian translation #86
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
base: main
Are you sure you want to change the base?
Conversation
|
@forkgenesis Thanks for your contribution! Unfortunately I can't (proof)read Russian, but otherwise it looks fine to me. As for whether or not to translate the keys, I have no strong opinion and would leave it to you to find a good solution. Other translations, such as Portuguese, have translated some but not all of the keys – maybe that makes sense here too. Some bare keys should certainly remain, which as of now unfortunately means basic Latin. We had hoped to fix that in v1.1, but it now seems that such a fix will arrive, at the earliest, in v1.2 🙁 . |
|
|
@forkgenesis No I don't think there's any need to wait for v1.1; having another translation of v1.0 is good in any case. @arp242 @pradyunsg What do you think? Is this good to merge as it is, or would you suggest changes? |
|
Any objections to merging this? If not, I'll do it in three days or so. |
|
I don't speak Russian. I can't even build the website. So I have no idea and can't check anything. |
|
Sorry if you got bombarded with notifications. I added another commit which fixed some lexical and grammatical errors, translated a couple of texts that hadn't been translated earlier, and translated some more code where it wouldn't interfere with the study of the specification. |
|
@arp242 I don't speak Russian either, but I did some quick checks and the translation seems basically okay, so I'd tend to merge it unless there's explicit opposition. As for building the website: Well I must admit I have no idea either. I could merge this but I don't know whether that'll automatically trigger a regeneration of the website or else how to trigger it. @pradyunsg can you tell us? |
|
What we've done in the past is asked around for another native speaker (in addition to the translator) to step forward and review the translation. |
|
IIRC, this repo is setup with Netlify and it'll rebuild when the PR is merged. I'd suggest checking past PRs, since AFAIK we didn't need to change anything outside of the repository to add new translations. |
|
Russian is my mother language. I'm probably not on a level of someone living full-time in Russia, not up to date with modern slang, but good enough to read average books. So I'll take a stab at this. Here is a preview page from Netlify that I've found |
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.
I think this is a great start.
But some translation is one to one from English, but sounds kinda weird in Russian, while technically being correct. I suggest we should apply some artistic translation here to improve readability.
I added couple of nits for consideration.
| <h3 | ||
| class="mt-12 mb-4 font-semibold text-4xl text-left text-orange-900 leading-none tracking-tight" | ||
| > | ||
| Формат файлов конфигурации <br class="md:hidden" />для людей. |
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.
nitpick: alternative suggestion
"Формат конфигурационных файлов для людей"
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.
Я использовал существительное вместо прилагательного, чтобы избежать больших (страшных) слов в главном заголовке на странице. Однако сейчас я вижу, что это может привести к непониманию, к чему относится обстоятельство "для людей". Поэтому исправлю
| /> | ||
| </svg> | ||
| </div> | ||
| TOML приоритизирует людей |
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.
Выглядит как перевод один в один с английского, но читается немного странно. "TOML для людей" звучит для меня лучше. Что думаешь?
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.
Да, здесь я не мог придумать ничего лучше. Спасибо, исправлю.
| <span class="font-bold">однозначно переводится</span> в хэш-таблицу | ||
| </li> | ||
| <li class="leading-snug mt-2"> | ||
| <span class="font-bold">легко парсить</span> в структуры данных в широком ряде языков |
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.
| <span class="font-bold">легко парсить</span> в структуры данных в широком ряде языков | |
| <span class="font-bold">легко парсится</span> в структуры данных в широком ряде языков |
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.
Непоследовательность с моей стороны. Сам же писал ранее, что "должен легко парситься".
| /> | ||
| </svg> | ||
| </div> | ||
| TOML широко поддерживается |
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.
Как то странно читается, хочется предложить альтернативные варианты.
- "TOML имеет широкую поддержку"
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.
Спасибо, учту и это!
| </dt> | ||
| <dd class="text-gray-600 ml-13"> | ||
| <p class="ml-1"> | ||
| У TOML уже существуют имплементации в большинстве самых популярных языков программирования |
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.
"У TOML уже существуют имплементации в большинстве самых популярных языков программирования для использования на сегодняшний день" => "У TOML уже существуют имплементации в большинстве самых популярных языках программирования"
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.
Не согласен. "в большинстве" требует родительного падежа зависимого слова. "в большинстве" ---кого?--> "языков". "Языках" - это предложный падеж слова "языки".
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.
Однако, я заметил у себя другую ошибку - "для использования" здесь не нужно, т.к. "самых популярных <..> программирования" уже предполагает собой популярность в использовании.
| <div | ||
| class="pt-12 pb-16 px-8 text-center border-orange-100 border-b-2 lg:flex-1 lg:py-32 lg:border-r-2" | ||
| > | ||
| <h3 class="text-2xl font-bold lg:text-3xl">Больше Спецификации</h3> |
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.
"Больше спецификации" как то дословно переведено ;) Может "Полная спецификация"?
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.
Вы правы! Немедленно исправлю. Спасибо!
| > | ||
| <h3 class="text-2xl font-bold lg:text-3xl">Больше Спецификации</h3> | ||
| <p class="text-gray-600"> | ||
| <strong>TOML</strong> поддерживает ещё больше родных типов и синтакса, прочтите об этом здесь: |
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.
| <strong>TOML</strong> поддерживает ещё больше родных типов и синтакса, прочтите об этом здесь: | |
| <strong>TOML</strong> имеет широкую поддержку разных типов и синтаксиса, прочитать об этом можно здесь: |
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.
Без сомнений, лучший вариант.
|
Thank you, Stas! I'll update the translation accordingly and then check it once more if some sections do need some artistic approach. The team of TOML, please, don't merge this PR for now <3 |
Hello! My name is Alex. I humbly request to add my translations of
index.htmlandv1.0.0.mdinto Russian language.I encourage any Russian-speaking contributors to proofread my work.
I also request opinions about whether all the keys should be russified or not. On one hand, it would improve the understanding of the examples given in the specification to the people who are not very familiar with the English language. On the other hand, it would clutter examples with lots of quotes and rid readers of the joy of bare keys.
Thank you, TOML team, for your awesome work. I hope my contibution will be useful to the community. <3
По-русски:
Привет! Меня зовут Алексей. Я прошу вас добавить мой перевод `index.html` и `v1.0.0.md` на русский язык. Я призываю всех русскоговорящих участников вычитать мою работу.Также, я прошу вашего мнения по поводу того, стоит ли русифицировать все ключи или нет. С одной стороны, это улучшит понимание примеров, данных в спецификации, людям, которые не понимают английского языка. С другой стороны, это засорит примеры кучей кавычек и избавит читателей от радости, которой являются простые ключи.
Спасибо вам, команда TOML, за вашу замечательную работу. Надеюсь, мой вклад будет полезным сообществу. <3