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

[I18N] Sorting order is incorrect in Switch language dropdown menu #7516

Closed
Zhuyun opened this issue Apr 15, 2014 · 7 comments · Fixed by #7593
Closed

[I18N] Sorting order is incorrect in Switch language dropdown menu #7516

Zhuyun opened this issue Apr 15, 2014 · 7 comments · Fixed by #7593

Comments

@Zhuyun
Copy link

Zhuyun commented Apr 15, 2014

Steps:

  1. Launch Brackets
  2. Debug > Switch Language

Result:
Sorting order of languages' name is incorrect

Expected:
Name of languages should sort by alphabet

Env: Win & Mac
Build: 0.38.0.12606

Please find screenshot below:
enu_edgecodep8_lk9 10_win_38_12606

@Zhuyun Zhuyun added the IQE label Apr 15, 2014
@marcelgerber
Copy link
Contributor

It's sorted via the ISO 3166-1 language codes (like de, fr, jp and it).
The problem about other sorting methods would possibly that non-latin chars are listed last.

@pthiess
Copy link
Contributor

pthiess commented Apr 21, 2014

I think we should close it as NAB. Thoughts?

@pthiess
Copy link
Contributor

pthiess commented Apr 21, 2014

No Priority - starter bug. This is a bug but since it's not an official feature we wouldn't spend time on it any time soon.

@marcelgerber
Copy link
Contributor

@pthiess You're saying this is a bug, so which order would you expect and where to place the languages beginning with non-latin chars?

@njx
Copy link

njx commented Apr 21, 2014

It seems like sorting them in some sort of lexicographic order by the display name (maybe Unicode order?) would make more sense than just having them be sorted by the underlying 2-char-code name, which isn't visible to the user. Even just having the ones that start with non-latin chars be in alphabetic order might make sense.

@marcelgerber
Copy link
Contributor

Fix is up (#7593)

@redmunds
Copy link
Contributor

Confirmed. Closing.

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

Successfully merging a pull request may close this issue.

5 participants