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

Partially revert "DolphinQt/Assembler: improve translatability" #12511

Merged

Conversation

JosJuice
Copy link
Member

This reverts the parts of commit c8c9928 that made translatability worse rather than better. Changing "Error in column %2" to "%1 in column %2" not only means that the translators have to check the i18n comments to know what word hides behind %1, but there's also the problem that the translator might need to translate "Error" in this context differently from the standalone string "Error". Having to copy-paste some HTML tags may be annoying for translators, but it's a far less serious problem.

This reverts the parts of commit c8c9928 that made translatability
worse rather than better. Changing "Error in column %2" to "%1 in column
%2" not only means that the translators have to check the i18n comments
to know what word hides behind %1, but there's also the problem that
the translator might need to translate "Error" in this context
differently from the standalone string "Error". Having to copy-paste
some HTML tags may be annoying for translators, but it's a far less
serious problem.
@Tilka Tilka merged commit 1a57040 into dolphin-emu:master Jan 20, 2024
11 checks passed
@JosJuice JosJuice deleted the partial-revert-assembler-translatability branch January 20, 2024 14:56
@JosJuice
Copy link
Member Author

For reference, this is how the Transifex editor handles these kinds of tags:

image

You just click on the tag and it copies the tag to the text box.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants