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

[HOLD] Sort project list alphabetically #8977

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

[HOLD] Sort project list alphabetically #8977

Changes from 6 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
0e2e2bd
Merge pull request #1 from adobe/master
brunnopleffken Aug 29, 2014
d172d75
Merge pull request #2 from adobe/master
brunnopleffken Sep 4, 2014
7a70914
Sort project list alphabetically
brunnopleffken Sep 5, 2014
482f58b
Merge branch 'master' of https://github.com/brunnopleffken/brackets
brunnopleffken Sep 5, 2014
a5cabac
Sort project list alphabetically
brunnopleffken Sep 5, 2014
6415f90
Sort project list alphabetically
brunnopleffken Sep 5, 2014
863a369
Sort project list alphabetically
brunnopleffken Sep 5, 2014
File filter

Filter by extension

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

templateVars.projectList.sort(function(a, b) {
// Changing the case (upper or lower) ensures a case insensitive sort
var aFolder = a.folder.toLocaleLowerCase();
var bFolder = b.folder.toLocaleLowerCase();
return aFolder.localeCompare(bFolder, brackets.getLocale());
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We usually use the options (3rd param) {numeric: true}

});

return Mustache.render(ProjectsMenuTemplate, templateVars);
}

Expand Down