diff options
author | Christopher Speller <crspeller@gmail.com> | 2015-06-30 14:06:00 -0400 |
---|---|---|
committer | Christopher Speller <crspeller@gmail.com> | 2015-06-30 14:06:00 -0400 |
commit | 7a7be75e742603d06927a488b136e1ed4432aac2 (patch) | |
tree | def708ab1cc29bb8b857d9eb9faafee06ae999b4 /web | |
parent | aa023c8b08171d74b4e4d48a82927b0702a86a01 (diff) | |
parent | 4022ab5ce0b8f8480ea2a6fb32d68422aa17e90e (diff) | |
download | chat-7a7be75e742603d06927a488b136e1ed4432aac2.tar.gz chat-7a7be75e742603d06927a488b136e1ed4432aac2.tar.bz2 chat-7a7be75e742603d06927a488b136e1ed4432aac2.zip |
Merge pull request #109 from nickago/MM-1289
Mm 1289 Now sorts extra channel members lexigraphically
Diffstat (limited to 'web')
-rw-r--r-- | web/react/components/channel_header.jsx | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/web/react/components/channel_header.jsx b/web/react/components/channel_header.jsx index 428d3ed81..48cb4d13b 100644 --- a/web/react/components/channel_header.jsx +++ b/web/react/components/channel_header.jsx @@ -44,10 +44,17 @@ var ExtraMembers = React.createClass({ var count = this.props.members.length == 0 ? "-" : this.props.members.length; count = this.props.members.length > 19 ? "20+" : count; var data_content = ""; + var sortedMembers = this.props.members; - this.props.members.forEach(function(m) { - data_content += "<div style='white-space: nowrap'>" + m.username + "</div>"; - }); + if(sortedMembers) { + sortedMembers.sort(function(a,b) { + return a.username.localeCompare(b.username); + }) + + sortedMembers.forEach(function(m) { + data_content += "<div style='white-space: nowrap'>" + m.username + "</div>"; + }); + } return ( <div style={{"cursor" : "pointer"}} id="member_popover" data-toggle="popover" data-content={data_content} data-original-title="Members" > |