diff --git a/public/less/admin/admin.less b/public/less/admin/admin.less index aa5efff256..23f3bead8c 100644 --- a/public/less/admin/admin.less +++ b/public/less/admin/admin.less @@ -146,7 +146,7 @@ body { color: @gray-dark; &:hover, &.selected { - background: black; + background: @brand-primary; color: white; } } diff --git a/public/less/admin/manage/groups.less b/public/less/admin/manage/groups.less index f0de916162..57057cc9ff 100644 --- a/public/less/admin/manage/groups.less +++ b/public/less/admin/manage/groups.less @@ -5,10 +5,25 @@ max-height: 500px; display: block; overflow-y: auto; - padding-bottom: 100px; .member-name { width: 100%; } } } -} \ No newline at end of file + + #group-icon { + cursor: pointer; + } +} + +.groups { + #group-search { + margin-bottom: 10px; + } + + .groups-list { + p { + margin: 0; + } + } +} diff --git a/public/src/admin/manage/group.js b/public/src/admin/manage/group.js index a64ca9dd63..05337f9f97 100644 --- a/public/src/admin/manage/group.js +++ b/public/src/admin/manage/group.js @@ -7,10 +7,10 @@ define('admin/manage/group', [ 'admin/modules/colorpicker', 'translator' ], function (memberList, iconSelect, colorpicker, translator) { - var Groups = {}; + var Groups = {}; Groups.init = function () { - var groupDetailsSearch = $('#group-details-search'), + var groupDetailsSearch = $('#group-details-search'), groupDetailsSearchResults = $('#group-details-search-results'), groupIcon = $('#group-icon'), changeGroupUserTitle = $('#change-group-user-title'), @@ -41,9 +41,12 @@ define('admin/manage/group', [ var searchText = groupDetailsSearch.val(), foundUser; - socket.emit('admin.user.search', {query: searchText}, function (err, results) { + socket.emit('admin.user.search', { + query: searchText + }, function (err, results) { if (!err && results && results.users.length > 0) { - var numResults = results.users.length, x; + var numResults = results.users.length, + x; if (numResults > 20) { numResults = 20; } @@ -53,7 +56,8 @@ define('admin/manage/group', [ for (x = 0; x < numResults; x++) { foundUser = $('
  • '); foundUser - .attr({title: results.users[x].username, + .attr({ + title: results.users[x].username, 'data-uid': results.users[x].uid, 'data-username': results.users[x].username, 'data-userslug': results.users[x].userslug, @@ -96,7 +100,12 @@ define('admin/manage/group', [ "icon:text": userLabel.attr('data-usericon-text') }; - templates.parse('partials/groups/memberlist', 'members', {group: {isOwner: ajaxify.data.group.isOwner, members: [member]}}, function (html) { + templates.parse('admin/partials/groups/memberlist', 'members', { + group: { + isOwner: ajaxify.data.group.isOwner, + members: [member] + } + }, function (html) { translator.translate(html, function (html) { $('[component="groups/members"] tbody').prepend(html); }); @@ -112,38 +121,38 @@ define('admin/manage/group', [ uid = userRow.attr('data-uid'), action = btnEl.attr('data-action'); - switch(action) { - case 'toggleOwnership': - socket.emit('groups.' + (isOwner ? 'rescind' : 'grant'), { - toUid: uid, + switch (action) { + case 'toggleOwnership': + socket.emit('groups.' + (isOwner ? 'rescind' : 'grant'), { + toUid: uid, + groupName: groupName + }, function (err) { + if (err) { + return app.alertError(err.message); + } + ownerFlagEl.toggleClass('invisible'); + }); + break; + + case 'kick': + bootbox.confirm('[[admin/manage/groups:edit.confirm-remove-user]]', function (confirm) { + if (!confirm) { + return; + } + socket.emit('admin.groups.leave', { + uid: uid, groupName: groupName }, function (err) { if (err) { return app.alertError(err.message); } - ownerFlagEl.toggleClass('invisible'); + userRow.slideUp().remove(); }); - break; - case 'kick': - bootbox.confirm('[[admin/manage/groups:edit.confirm-remove-user]]', function (confirm) { - if (!confirm) { - return; - } - socket.emit('admin.groups.leave', { - uid: uid, - groupName: groupName - }, function (err) { - if (err) { - return app.alertError(err.message); - } - userRow.slideUp().remove(); - }); - - }); - break; - default: - break; + }); + break; + default: + break; } }); @@ -155,7 +164,7 @@ define('admin/manage/group', [ groupLabelPreview.css('background-color', '#' + hex); }); - $('.save').on('click', function () { + $('#save').on('click', function () { socket.emit('admin.groups.update', { groupName: groupName, values: { diff --git a/src/views/admin/manage/group.tpl b/src/views/admin/manage/group.tpl index f119f51914..0805da3e06 100644 --- a/src/views/admin/manage/group.tpl +++ b/src/views/admin/manage/group.tpl @@ -39,7 +39,7 @@
    - -
    - -
    -
    -
    [[admin/manage/groups:control-panel]]
    -
    -
    -
    - -
    -
    - -
    -
    -
    -
    -
    - + + - diff --git a/src/views/admin/manage/groups.tpl b/src/views/admin/manage/groups.tpl index c2b0ca2f32..98e08be2e7 100644 --- a/src/views/admin/manage/groups.tpl +++ b/src/views/admin/manage/groups.tpl @@ -1,47 +1,45 @@ -
    -
    -
    -
    - - - - - - - - - - - -
    [[admin/manage/groups:name]][[admin/manage/groups:description]]
    - {groups.displayName} - - [[admin/manage/groups:system]] - - -
    - - [[admin/manage/groups:edit]] - - - - -
    -

    {groups.description}

    -
    - -
    +
    +
    +
    +
    -
    -
    -
    -
    -
    - -
    -
    -
    + + + + + + + + + + + + + + + + + +
    [[admin/manage/groups:name]]
    + {groups.displayName} + + [[admin/manage/groups:system]] + + +
    + + [[admin/manage/groups:edit]] + + + + +
    +
    + +
    -
    - diff --git a/src/views/admin/partials/groups/memberlist.tpl b/src/views/admin/partials/groups/memberlist.tpl new file mode 100644 index 0000000000..23ce3e1dae --- /dev/null +++ b/src/views/admin/partials/groups/memberlist.tpl @@ -0,0 +1,37 @@ +
    + + +

    + + + + + + + + + + +
    + + + + +
    {group.members.icon:text}
    + +     +     		+ {group.members.username} + + +
    + + + + + + + +
    + +
    \ No newline at end of file