Skip to content

Commit d3d799b

Browse files
committed
Fixed minor issue of SortBy button
1 parent d33cb67 commit d3d799b

3 files changed

Lines changed: 3 additions & 5 deletions

File tree

js/Settings.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,6 @@ module.exports = {
6262
this.EContactsPrimaryAddress = Types.pObject(oAppDataSection.PrimaryAddress);
6363
this.EContactSortField = Types.pObject(oAppDataSection.SortField);
6464
this.ContactsSortBy = this.getSortConfig(Types.pObject(oAppDataSection.ContactsSortBy));
65-
66-
console.log('ContactsSortBy', this.ContactsSortBy);
6765
}
6866
},
6967

js/views/CContactsView.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -423,7 +423,7 @@ CContactsView.prototype.executeSort = function (sValue)
423423
if (sCurrentSort === sValue) {
424424
this.sortOrder(this.sortOrder() === Enums.SortOrder.Asc ? Enums.SortOrder.Desc : Enums.SortOrder.Asc); // Asc: 0, Desc: 1
425425
} else {
426-
this.sortOrder(Enums.SortOrder[Settings.ContactsSortBy.DefaultSortOrder]);
426+
this.sortOrder(Settings.ContactsSortBy.DefaultSortOrder);
427427
}
428428

429429
this.requestContactList();

templates/Toolbar/SortButtonView.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
<span class="item sort" data-bind="dropdown: {control: false},
1+
<span class="item sort" data-bind="dropdown: {control: false}, css: {'sort_asc': sortOrder() === Enums.SortOrder.Asc, 'sort_desc': sortOrder() === Enums.SortOrder.Desc},
22
customTooltip: 'COREWEBCLIENT/ACTION_SORT_BY'">
33
<span class="icon"></span>
44
<span class="text" data-bind="i18n: {'key': 'COREWEBCLIENT/ACTION_SORT_BY'}"></span>
5-
<span class="icon arrow"></span>
5+
<!-- <span class="icon arrow"></span> -->
66
<span class="dropdown">
77
<span class="dropdown_helper">
88
<span class="dropdown_arrow"><span></span></span>

0 commit comments

Comments
 (0)