From 5feabd46340a288494d6ac12ef71aca8d6777f0b Mon Sep 17 00:00:00 2001 From: m4dz Date: Mon, 12 Jun 2017 10:13:52 +0200 Subject: [PATCH 1/2] [fix] Highlight 'everyone' group by default --- settings/js/users/groups.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/settings/js/users/groups.js b/settings/js/users/groups.js index 16621441a6..c0f738acaf 100644 --- a/settings/js/users/groups.js +++ b/settings/js/users/groups.js @@ -380,4 +380,7 @@ $(document).ready( function () { $('#newgroupname').on('input', function(){ GroupList.handleAddGroupInput(this.value); }); + + // highlight `everyone` group at DOMReady by default + GroupList.showGroup('_everyone'); }); From 56adbb4e1dab40cd2ae26e99451c2784eb5346f9 Mon Sep 17 00:00:00 2001 From: m4dz Date: Mon, 12 Jun 2017 11:19:31 +0200 Subject: [PATCH 2/2] [docs] remove useless TODO mark --- settings/js/users/groups.js | 1 - 1 file changed, 1 deletion(-) diff --git a/settings/js/users/groups.js b/settings/js/users/groups.js index c0f738acaf..073551c0d8 100644 --- a/settings/js/users/groups.js +++ b/settings/js/users/groups.js @@ -222,7 +222,6 @@ GroupList = { UserList.update(gid === '_everyone' ? '' : gid); $userGroupList.find('li').removeClass('active'); if (gid !== undefined) { - //TODO: treat Everyone properly GroupList.getGroupLI(gid).addClass('active'); } },