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

Sort languages via their display names #7593

Merged
merged 3 commits into from
Apr 23, 2014
Merged

Sort languages via their display names #7593

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
baedede
Sort languages via their display names
Apr 21, 2014
3ed9e70
Merge remote-tracking branch 'upstream/master' into fix-7516
Apr 23, 2014
52ce020
Remove unnecessary vars
Apr 23, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 7 additions & 6 deletions src/extensions/default/DebugCommands/main.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -167,12 +167,6 @@ define(function (require, exports, module) {

return i18n === undefined ? locale : i18n;
};

// add system default
languages.push({label: Strings.LANGUAGE_SYSTEM_DEFAULT, language: null});

// add english
languages.push({label: getLocalizedLabel("en"), language: "en"});

// inspect all children of dirEntry
entries.forEach(function (entry) {
Expand All @@ -191,6 +185,13 @@ define(function (require, exports, module) {
}
}
});
// sort the languages via their display name
languages.sort(function (lang1, lang2) {
return lang1.label.localeCompare(lang2.label);
});

// add system default and english (those should be on the very top)
languages.unshift({label: Strings.LANGUAGE_SYSTEM_DEFAULT, language: null}, {label: getLocalizedLabel("en"), language: "en"});

var template = Mustache.render(LanguageDialogTemplate, {languages: languages, Strings: Strings});
Dialogs.showModalDialogUsingTemplate(template).done(function (id) {
Expand Down