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

DolphinQt: Add i18n comment for "Keep addresses where value in memory" #10148

Merged
merged 1 commit into from Oct 3, 2021

Conversation

JosJuice
Copy link
Member

@JosJuice JosJuice commented Oct 3, 2021

This string is extremely likely to be mistranslated without the proper context. Actually, it's probably impossible to translate this string in a good way to some languages, but I'm not sure how to solve that. Let's at least add an i18 comment for now.

@AdmiralCurtiss
Copy link
Contributor

Maybe combining the label with the dropdown would make it better? But yeah I admit this is not particularly localization-friendly...

This string is extremely likely to be mistranslated without the
proper context. Actually, it's probably impossible to translate
this string in a good way to some languages, but I'm not sure how
to solve that. Let's at least add an i18 comment for now.
@JosJuice
Copy link
Member Author

JosJuice commented Oct 3, 2021

I'm not sure if there are any languages we support where that actually would help. My main concern is with languages where the verb comes at the end of the sentence.

@leoetlino leoetlino merged commit 9dc6229 into dolphin-emu:master Oct 3, 2021
10 checks passed
@JosJuice JosJuice deleted the i18n-keep-addresses branch October 3, 2021 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants