Skip to content

Conversation

brlin-tw
Copy link
Contributor

@brlin-tw brlin-tw commented Jun 29, 2024

This patch provides user interface string translations in Taiwanese Chinese(as far as the current I18N implementation allows).

There are some UI strings that isn't translatable probably due to they're not marked as resource strings, or belong to another package(pydroid):

Screenshot_20240629-162120
Screenshot_20240629-162156

@pyamsoft pyamsoft self-requested a review June 29, 2024 19:15
@pyamsoft pyamsoft added the enhancement New feature or request label Jun 29, 2024
@pyamsoft
Copy link
Owner

Thank you for your contribution!

I will look this PR over and hopefully have some feedback for you soon. As a full disclosure, since I personally do not read Taiwanese Chinese, I will have to take some extra time by passing these strings through Google Translate just to make sure that there is no language which is unintentionally hurtful or harmful.

Thank you!

@brlin-tw brlin-tw force-pushed the patch/implement-zhrtw-localization branch 2 times, most recently from eb564d1 to b8f7e09 Compare June 30, 2024 12:48
This patch provides user interface string translation in Taiwanese Chinese.
@brlin-tw
Copy link
Contributor Author

brlin-tw commented Jul 1, 2024

Note that currently there is a string interpolation error in the error details view:

Screenshot

@brlin-tw brlin-tw force-pushed the patch/implement-zhrtw-localization branch from b8f7e09 to a087141 Compare July 1, 2024 14:22
pyamsoft added a commit that referenced this pull request Jul 1, 2024
I guess even from a different resource file, the same named
string res ID can collide. Fun.

Blank out the title to avoid the issue and reduce noise.
Should address the comment in #310
@pyamsoft
Copy link
Owner

Hi, sorry this has been taking so long. I have not forgotten, I have just been busy. I hope to have it reviewed and merged soon, thanks again!

@pyamsoft pyamsoft merged commit 8fcb648 into pyamsoft:main Jul 27, 2024
@brlin-tw brlin-tw deleted the patch/implement-zhrtw-localization branch July 27, 2024 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants