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

i18n stuff #56

Merged
merged 1 commit into from
Aug 23, 2016
Merged

i18n stuff #56

merged 1 commit into from
Aug 23, 2016

Conversation

aaronj1335
Copy link
Contributor

  • add a utility for making a CSV of strings to make it easier to solicit
    translations
  • remove english from uk/ru strings files
  • fix a bug that resulted in parts of the app showing the system default
    strings, and parts not.
    • specifically, Locale.getDefault() returns a different locale after
      resources.updateConfiguration() is called. so if you switch to uk,
      then to the system default, Locale.getDefault() would return uk,
      resulting in the nav not going back to english. i understand that if
      the user changes the system language, this app won't reflect that
      change, but that seems less bad than a UI showing two different
      languages.

- add a utility for making a CSV of strings to make it easier to solicit
  translations
- remove english from uk/ru strings files
- fix a bug that resulted in parts of the app showing the system default
  strings, and parts not.
  - specifically, `Locale.getDefault()` returns a different locale after
    `resources.updateConfiguration()` is called. so if you switch to uk,
    then to the system default, `Locale.getDefault()` would return uk,
    resulting in the nav not going back to english. i understand that if
    the user changes the system language, this app won't reflect that
    change, but that seems less bad than a UI showing two different
    languages.
@joshganderson
Copy link
Contributor

LGTM

@aaronj1335 aaronj1335 merged commit b9431f2 into g11x:master Aug 23, 2016
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.

2 participants