diff --git a/submodules/users/users.css b/submodules/users/users.css index fc34841..fcbd54a 100644 --- a/submodules/users/users.css +++ b/submodules/users/users.css @@ -96,6 +96,7 @@ } #users_container .users-grid .grid-row.title .grid-cell.name , .users-grid .grid-row.title .grid-cell.extension { + cursor: pointer; font-weight: bold; } diff --git a/submodules/users/users.js b/submodules/users/users.js index 04a0cd4..038db74 100644 --- a/submodules/users/users.js +++ b/submodules/users/users.js @@ -424,16 +424,19 @@ define(function(require) { var sortedUsers = []; //Set blank presence_id for ability to sort by presence_id _.each(mapUsers, function(user) { - if(!user.hasOwnProperty('presence_id')){ + if (!user.hasOwnProperty('presence_id')) { user.presence_id = ''; } + sortedUsers.push(user); }); + //Default sort by presence_id - if(typeof _sortBy === 'undefined' ){ - _sortBy = "first_name"; - } - sortedUsers = self.sort(sortedUsers, _sortBy); + if (typeof _sortBy === 'undefined') { + _sortBy = 'first_name'; + } + + sortedUsers = self.sort(sortedUsers, _sortBy); dataTemplate.users = sortedUsers; @@ -452,10 +455,9 @@ define(function(require) { }, result; - if(typeof secondArg === 'function') { + if (typeof secondArg === 'function') { sortFunction = secondArg; - } - else if(typeof secondArg === 'string') { + } else if (typeof secondArg === 'string') { fieldName = secondArg; } @@ -530,7 +532,7 @@ define(function(require) { }); template.find('.grid-row.title .grid-cell.extension').on('click', function() { - self.usersRender({ sortBy: 'presence_id' }); + self.usersRender({ sortBy: 'presence_id' }); }); template.find('.grid-row:not(.title) .grid-cell').on('click', function() {