Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Locale description in switching language UI #1461

Closed
jbalsas opened this issue Aug 27, 2012 · 4 comments
Closed

Locale description in switching language UI #1461

jbalsas opened this issue Aug 27, 2012 · 4 comments

Comments

@jbalsas
Copy link
Contributor

jbalsas commented Aug 27, 2012

In the locale switching UI, it would be nice to see a description of the locale.

Right now, the language list gets populated using the locale codes. I know Brackets is a code editor and probably almost everyone knows about locale codes, but it would be nice to complement it with a description like "English (United States)" or "English (United Kingdom)" for instance.

@njx
Copy link
Contributor

njx commented Aug 27, 2012

Makes sense. I think this UI is just temporary though--eventually we'll have some sort of real prefs dialog with a sensible UI control (instead of clicking on a bulleted list :)). So we might not want to invest in fixing this right now.

@jbalsas
Copy link
Contributor Author

jbalsas commented Aug 28, 2012

I agree that it's not worth the effort to fix this for the UI right now.

What maybe is, is to think where this description should be. Right now there are only two translations (root and fr) but I'm working on the spanish one, and I've also seen a pull request for norwegian. If the description must be in nls/[locale]/string.js, I think it would be better to know now so the translators can get everything done in one pass and you won't have to be chasing them later :)

Anyway, this is not really an issue. Can you close it or should I?

@gruehle
Copy link
Member

gruehle commented Aug 28, 2012

Reviewed. Moving to backlog as a new feature. Thanks for filing!

@pthiess
Copy link
Contributor

pthiess commented Oct 25, 2012

Created Trello Card #675, closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants