diff --git a/apps/common/main/lib/controller/Comments.js b/apps/common/main/lib/controller/Comments.js index 89cba715a..a93ccca3c 100644 --- a/apps/common/main/lib/controller/Comments.js +++ b/apps/common/main/lib/controller/Comments.js @@ -210,19 +210,24 @@ define([ if (this.collection) { var sort = (type !== undefined); if (type === undefined) { - type = Common.localStorage.getItem(this.appPrefix + "comments-sort") || 'date'; + type = Common.localStorage.getItem(this.appPrefix + "comments-sort") || 'date-desc'; } Common.localStorage.setItem(this.appPrefix + "comments-sort", type); Common.Utils.InternalSettings.set(this.appPrefix + "comments-sort", type); if (type=='position') { - } else if (type=='author') { - this.collection.comparator = function (collection) { - return collection.get('parsedName').toLowerCase(); + } else if (type=='author-asc' || type=='author-desc') { + var direction = (type=='author-asc') ? 1 : -1; + this.collection.comparator = function(item1, item2) { + var n1 = item1.get('parsedName').toLowerCase(), + n2 = item2.get('parsedName').toLowerCase(); + if (n1==n2) return 0; + return (n1
To copy or paste to or from applications outside the editor tab use the following keyboard combinations:", diff --git a/apps/presentationeditor/main/locale/en.json b/apps/presentationeditor/main/locale/en.json index 4129c9812..29eb081a2 100644 --- a/apps/presentationeditor/main/locale/en.json +++ b/apps/presentationeditor/main/locale/en.json @@ -141,8 +141,10 @@ "Common.Views.Comments.textResolved": "Resolved", "Common.Views.Comments.textSort": "Sort comments", "Common.Views.Comments.mniPosition": "Sort by Position", - "Common.Views.Comments.mniAuthor": "Sort by Authors", - "Common.Views.Comments.mniDate": "Sort by Date", + "Common.Views.Comments.mniAuthorAsc": "Sort by Author A to Z", + "Common.Views.Comments.mniAuthorDesc": "Sort by Author Z to A", + "Common.Views.Comments.mniDateDesc": "Sort by Newest", + "Common.Views.Comments.mniDateAsc": "Sort by Oldest", "Common.Views.Comments.textClosePanel": "Close comments", "Common.Views.CopyWarningDialog.textDontShow": "Don't show this message again", "Common.Views.CopyWarningDialog.textMsg": "Copy, cut and paste actions using the editor toolbar buttons and context menu actions will be performed within this editor tab only.

To copy or paste to or from applications outside the editor tab use the following keyboard combinations:", diff --git a/apps/spreadsheeteditor/main/locale/en.json b/apps/spreadsheeteditor/main/locale/en.json index e0bcb8240..6a72a95fc 100644 --- a/apps/spreadsheeteditor/main/locale/en.json +++ b/apps/spreadsheeteditor/main/locale/en.json @@ -189,8 +189,10 @@ "Common.Views.Comments.textResolved": "Resolved", "Common.Views.Comments.textSort": "Sort comments", "Common.Views.Comments.mniPosition": "Sort by Position", - "Common.Views.Comments.mniAuthor": "Sort by Authors", - "Common.Views.Comments.mniDate": "Sort by Date", + "Common.Views.Comments.mniAuthorAsc": "Sort by Author A to Z", + "Common.Views.Comments.mniAuthorDesc": "Sort by Author Z to A", + "Common.Views.Comments.mniDateDesc": "Sort by Newest", + "Common.Views.Comments.mniDateAsc": "Sort by Oldest", "Common.Views.Comments.textClosePanel": "Close comments", "Common.Views.CopyWarningDialog.textDontShow": "Don't show this message again", "Common.Views.CopyWarningDialog.textMsg": "Copy, cut and paste actions using the editor toolbar buttons and context menu actions will be performed within this editor tab only.

To copy or paste to or from applications outside the editor tab use the following keyboard combinations:",