diff options
author | Christopher Speller <crspeller@gmail.com> | 2015-10-16 10:15:52 -0400 |
---|---|---|
committer | Christopher Speller <crspeller@gmail.com> | 2015-10-16 10:50:10 -0400 |
commit | 495673d80d75ddfc97dc8d9fe61021142418dcc2 (patch) | |
tree | 3c7bba0806c89ab85faa9ffecd1322e22569370a /web/react/components/popover_list_members.jsx | |
parent | 3c593af598b2bb986244099ed1b9a46ed3837911 (diff) | |
download | chat-495673d80d75ddfc97dc8d9fe61021142418dcc2.tar.gz chat-495673d80d75ddfc97dc8d9fe61021142418dcc2.tar.bz2 chat-495673d80d75ddfc97dc8d9fe61021142418dcc2.zip |
Fixing some client warnings
Diffstat (limited to 'web/react/components/popover_list_members.jsx')
-rw-r--r-- | web/react/components/popover_list_members.jsx | 24 |
1 files changed, 19 insertions, 5 deletions
diff --git a/web/react/components/popover_list_members.jsx b/web/react/components/popover_list_members.jsx index 16ae693fa..155e88600 100644 --- a/web/react/components/popover_list_members.jsx +++ b/web/react/components/popover_list_members.jsx @@ -35,13 +35,20 @@ export default class PopoverListMembers extends React.Component { const teamMembers = UserStore.getProfilesUsernameMap(); if (members && teamMembers) { - members.sort(function compareByLocal(a, b) { + members.sort((a, b) => { return a.username.localeCompare(b.username); }); - members.forEach(function addMemberElement(m) { + members.forEach((m, i) => { if (teamMembers[m.username] && teamMembers[m.username].delete_at <= 0) { - popoverHtml.push(<div className='text--nowrap'>{m.username}</div>); + popoverHtml.push( + <div + className='text--nowrap' + key={'popover-member-' + i} + > + {m.username} + </div> + ); count++; } }); @@ -57,8 +64,15 @@ export default class PopoverListMembers extends React.Component { <OverlayTrigger trigger='click' placement='bottom' - rootClose='true' - overlay={<Popover title='Members'>{popoverHtml}</Popover>} + rootClose={true} + overlay={ + <Popover + title='Members' + id='member-list-popover' + > + {popoverHtml} + </Popover> + } > <div id='member_popover'> <div> |