-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Add Language Chooser #2150
Add Language Chooser #2150
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Switching the language is a nice to have feature.
Although: i don't like that a language needs to be added manually. Can't we generate some kind of properties file during build and load it via the getResource(String name)
? Then we would notice when a new language is added.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, and please adjust the SettingsJsonAdapterTest
to also include the language tag.
Tests for SupportedLanguages
would also be nice 😉
me neither, but I wanted to keep the PR small, since the last one was so big. |
Kudos, SonarCloud Quality Gate passed! |
Fixes #2106
Important: When adding a new language, it needs to be added to
SupportedLanguages#LANGUAGAE_TAGS
manually in order to show up in the preferences menu.