这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@HarGabt
Copy link
Contributor

@HarGabt HarGabt commented Aug 31, 2022

Adding the localisation feature is a welcome addition to this project - something I as a localiser had been excited for a long time. Alas, the strings introduced in this commit give the feeling of being google-translated. So...

Most of strings were renamed according to the official localisation (bundled with Russian WinXP distros, manual only), some of them I had to adapt myself.

Feel free to correct some of those strings and switch to Russian language in this conversation if needed.

Most of strings were renamed according to the official localisation (bundled with russian WinXP distros, manual only), some of them I had to adapt myself.
@k4zmu2a
Copy link
Owner

k4zmu2a commented Sep 1, 2022

It is about time someone with proper language background corrected my mess.

Russian texts you see were not in fact done using machine translation)
I added them as part of the first localization test in v1. At that time, it did not occur to me to read the chm for guidance.
Later, I considered wiping it in favor of authentic English texts used in Ru version but decided against it in the end.

The only concern I have about new texts is with “wormhole” == “норка”.
In my opinion, “червоточина” seems less ambiguous for a space-themed game.
Does this term come from the official chm?

I prefer English for public discussions about the project.

@HarGabt
Copy link
Contributor Author

HarGabt commented Sep 1, 2022

Does this term come from the official chm?

Picture behind a spoiler (Yes, it does come from the official .chm file)

изображение

Your choice is your choice to put Червоточина instead of Норка

@k4zmu2a k4zmu2a merged commit 69fd91f into k4zmu2a:master Sep 2, 2022
@k4zmu2a
Copy link
Owner

k4zmu2a commented Sep 2, 2022

Thank you, merged.

@win98se
Copy link

win98se commented Sep 20, 2022

I think the translation improvements in here and also #156 can be merged into v1 as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants