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

SettingsScreen: Translate languages' names into selected language #5843

Merged
merged 2 commits into from
Mar 7, 2024

Conversation

chrisbobbe
Copy link
Contributor

No description provided.

@chrisbobbe chrisbobbe added the a-i18n Translation, localization, internationalization label Mar 7, 2024
@chrisbobbe chrisbobbe requested a review from gnprice March 7, 2024 00:50
These used to be translated, but then we accidentally stopped in
a483f56 about a year ago. Oops; fix.
It looks odd to have an item where the title and subtitle are
slightly different ways of naming the same thing. The canonical
self-name is the one we've chosen by following organizations
with a broad global contributor base like Wikipedia, so use that.

(We could instead just omit the subtitle, but if we did, then all
the rows in the list would have two lines of text except the
selected row which would have just one. And that non-uniformity
might look odd.)
@gnprice
Copy link
Member

gnprice commented Mar 7, 2024

Thanks! Merged, with some commit-message tweaks.

@gnprice gnprice merged commit 630caee into zulip:main Mar 7, 2024
1 check passed
@chrisbobbe chrisbobbe deleted the pr-language-names branch April 10, 2024 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-i18n Translation, localization, internationalization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants