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

Ability to select app language #901

Merged
merged 28 commits into from
Nov 24, 2023
Merged

Conversation

hjiangsu
Copy link
Member

@hjiangsu hjiangsu commented Nov 17, 2023

Pull Request Description

This is a PR which introduces the ability to select Thunder's localization language from the Settings page.

This PR changes/introduces a couple new things:

  • It sorts all the .arb files to be alphabetical in order. When adding new translation strings, attempts should be made to ensure it stays in alphabetical order. I have locked Weblate translations temporarily until this is merged in and Weblate is synced up with the .arb changes.
  • A new LanguageLocal class was added in to help translate language codes to the respective names
  • Added a new package l10n_esperanto to help with the Esperanto localization. Android does not natively support this, so this localization is not part of the default flutter localization packages.

Issue Being Fixed

Issue Number: #899

Screenshots / Recordings

translation_setting.mp4

Checklist

  • Did you update CHANGELOG.md?
  • Did you use localized strings where applicable?
  • Did you add semanticLabels where applicable for accessibility?

pioneer and others added 26 commits November 24, 2023 17:59
Currently translated at 32.0% (85 of 265 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/uk/
Currently translated at 100.0% (269 of 269 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/es/
Currently translated at 100.0% (272 of 272 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/es/
Currently translated at 23.2% (64 of 275 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/sv/
Currently translated at 100.0% (280 of 280 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/es/
Currently translated at 50.0% (140 of 280 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/sv/
Currently translated at 50.0% (140 of 280 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/sv/
Currently translated at 100.0% (280 of 280 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/eo/
Currently translated at 77.5% (217 of 280 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/pl/
Currently translated at 30.7% (86 of 280 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/uk/
Currently translated at 46.4% (130 of 280 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/fi/
Currently translated at 15.0% (42 of 280 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/it/
Currently translated at 38.2% (107 of 280 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/it/
Currently translated at 55.7% (156 of 280 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/it/
Currently translated at 68.5% (192 of 280 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/it/
Currently translated at 100.0% (280 of 280 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/es/
Currently translated at 100.0% (280 of 280 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/eo/
Currently translated at 90.7% (254 of 280 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/it/
Currently translated at 100.0% (282 of 282 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/es/
Currently translated at 90.7% (256 of 282 strings)

Translation: Thunder/Thunder
Translate-URL: https://hosted.weblate.org/projects/thunder/thunder/it/
@hjiangsu hjiangsu force-pushed the feature/app-language-selection branch from 1a43d5c to 834af59 Compare November 24, 2023 19:24
@hjiangsu hjiangsu marked this pull request as ready for review November 24, 2023 19:37
@hjiangsu
Copy link
Member Author

@micahmo This is ready for review if you'd like to do a quick test on it!

Copy link
Member

@micahmo micahmo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

We might have some RTL issues, but that's separate. 😊

Are we missing a language in the UI?

image

image

@hjiangsu
Copy link
Member Author

Are we missing a language in the UI?

Good catch, let me double check that!

@hjiangsu
Copy link
Member Author

Should be good now! I double all the files with https://github.com/thunder-app/thunder/tree/translations/lib/l10n to just make sure we have all the translations in place.

@hjiangsu hjiangsu merged commit b358b8a into develop Nov 24, 2023
1 check passed
@hjiangsu hjiangsu deleted the feature/app-language-selection branch November 24, 2023 20:22
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.

9 participants