Limit amount of avatars shown #217

BeagleLab · Jul 15, 2015 · f8fe3bd · f8fe3bd
1 parent 489bbba
commit f8fe3bd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/build/main.min.js b/build/main.min.js
@@ -591,9 +591,7 @@ module.exports = exports = React.createClass({
     return React.createElement('div', null, this.state.conversations.map(function (conversation) {
       var avatars = conversation.avatars;
 
-      return React.createElement('div', { style: listingStyle }, React.createElement('p', { style: titleStyle }, conversation.title), React.createElement('div', { style: imageWrapperStyle },
-      /* Todo: Limit to 8 or so avatars */
-      conversation.avatars.map(function (avatar) {
+      return React.createElement('div', { style: listingStyle }, React.createElement('p', { style: titleStyle }, conversation.title), React.createElement('div', { style: imageWrapperStyle }, conversation.avatars.slice(0, 5).map(function (avatar) {
         return React.createElement('img', { style: imgStyle, key: avatars.indexOf(avatar), src: avatar });
       })), React.createElement('span', { style: dateStyle }, 'Updated ', conversation.date, ' ago'), React.createElement('div', { style: commentStyle }, React.createElement('i', { className: 'fa fa-comment' }), ' ', conversation.notes.length));
     }));

diff --git a/js/components/conversations.jsx b/js/components/conversations.jsx
@@ -53,8 +53,7 @@ module.exports = exports = React.createClass({
               <div style={listingStyle}>
                 <p style={titleStyle}>{conversation.title}</p>
                 <div style={imageWrapperStyle}>
-                  {/* Todo: Limit to 8 or so avatars */}
-                  {conversation.avatars.map(function (avatar) {
+                  {conversation.avatars.slice(0, 5).map(function (avatar) {
                     return <img style={imgStyle} key={avatars.indexOf(avatar)} src={avatar} />
                   })}
                 </div>