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

[CLOSED] Sort languages via their display names #6836

Open
core-ai-bot opened this issue Aug 30, 2021 · 10 comments
Open

[CLOSED] Sort languages via their display names #6836

core-ai-bot opened this issue Aug 30, 2021 · 10 comments

Comments

@core-ai-bot
Copy link
Member

Issue by MarcelGerber
Monday Apr 21, 2014 at 19:32 GMT
Originally opened as adobe/brackets#7593


This fixes #7516.


MarcelGerber included the following code: https://github.com/adobe/brackets/pull/7593/commits

@core-ai-bot
Copy link
Member Author

Comment by redmunds
Tuesday Apr 22, 2014 at 23:13 GMT


Is there a reason that italiano, norsk, polski, and suomi are not capitalized?

@core-ai-bot
Copy link
Member Author

Comment by TomMalbran
Tuesday Apr 22, 2014 at 23:48 GMT


I think that English should be sorted to, so that we have English (US) and English (UK) together.

@core-ai-bot
Copy link
Member Author

Comment by MarcelGerber
Wednesday Apr 23, 2014 at 01:31 GMT


@redmunds Looks like it's intended to do so - take a look at https://www.google.com/preferences#languages.

@TomMalbran Don't know what to do here, and English (UK) does not yet much in Brackets (basically nothing...).

@core-ai-bot
Copy link
Member Author

Comment by redmunds
Wednesday Apr 23, 2014 at 04:12 GMT


@SAPlayer

Looks like it's intended to do so - take a look at https://www.google.com/preferences#languages.

I don't think that's a good example, since some of the ones that we have capitalized are lowercase there. It's not really related to this change, so we don't need to worry about it.

@core-ai-bot
Copy link
Member Author

Comment by MarcelGerber
Wednesday Apr 23, 2014 at 12:52 GMT


@redmunds Changes pushed. We should consider looking at the locale names a second time, Google and Wikipedia state other names sometimes (or other casing). I could do that if you want.

@core-ai-bot
Copy link
Member Author

Comment by redmunds
Wednesday Apr 23, 2014 at 14:26 GMT


@TomMalbran

I think that English should be sorted to, so that we have English (US) and English (UK) together.

This dialog was designed to help those translating from English (US) to some other language, so that's why it was placed at the top. This PR is about ordering the other items in the list, so we'll just keep it there for now.

@core-ai-bot
Copy link
Member Author

Comment by redmunds
Wednesday Apr 23, 2014 at 14:29 GMT


Looks good. Merging.

@core-ai-bot
Copy link
Member Author

Comment by TomMalbran
Wednesday Apr 23, 2014 at 16:11 GMT


In that case, we could place the other English locales at the top too?

@core-ai-bot
Copy link
Member Author

Comment by MarcelGerber
Wednesday Apr 23, 2014 at 18:00 GMT


Well, at least for now English (UK) doesn't differ from English (US) in core. It would be nice to see them beneath each other, but that would possibly require hardcoding en-gb to the top, which isn't the best.

@core-ai-bot
Copy link
Member Author

Comment by peterflynn
Thursday Apr 24, 2014 at 01:21 GMT


@TomMalbran@SAPlayer Actually, do you know why we bother forcing "English (US)" to the top of the list? From a user standpoint, there's nothing special about "English (US)" -- users don't care that it's the "root" language under the hood... (But of course I can see why we'd want "System Default" at the top -- definitely not suggesting changing that).

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

No branches or pull requests

1 participant