From ca53a63fd8150ced38c9367a2b12a92f3f88f4f1 Mon Sep 17 00:00:00 2001
From: Toby Zerner <toby.zerner@gmail.com>
Date: Tue, 8 Sep 2015 10:27:50 +0930
Subject: [PATCH] Make unread discussion titles less overwhelming

---
 framework/core/js/forum/src/components/DiscussionListItem.js | 3 ++-
 framework/core/js/lib/models/Discussion.js                   | 3 ++-
 framework/core/less/forum/DiscussionListItem.less            | 5 ++++-
 3 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/framework/core/js/forum/src/components/DiscussionListItem.js b/framework/core/js/forum/src/components/DiscussionListItem.js
index 045357b65..f82cbb32d 100644
--- a/framework/core/js/forum/src/components/DiscussionListItem.js
+++ b/framework/core/js/forum/src/components/DiscussionListItem.js
@@ -51,6 +51,7 @@ export default class DiscussionListItem extends Component {
     const discussion = this.props.discussion;
     const startUser = discussion.startUser();
     const isUnread = discussion.isUnread();
+    const isRead = discussion.isRead();
     const showUnread = !this.showRepliesCount() && isUnread;
     const jumpTo = Math.min(discussion.lastPostNumber(), (discussion.readNumber() || 0) + 1);
     const relevantPosts = this.props.params.q ? discussion.relevantPosts() : [];
@@ -71,7 +72,7 @@ export default class DiscussionListItem extends Component {
           {icon('check')}
         </a>
 
-        <div className={'DiscussionListItem-content Slidable-content' + (isUnread ? ' unread' : '')}>
+        <div className={'DiscussionListItem-content Slidable-content' + (isUnread ? ' unread' : '') + (isRead ? ' read' : '')}>
           <a href={startUser ? app.route.user(startUser) : '#'}
             className="DiscussionListItem-author"
             title={extractText(app.trans('core.discussion_started', {user: startUser, ago: humanTime(discussion.startTime())}))}
diff --git a/framework/core/js/lib/models/Discussion.js b/framework/core/js/lib/models/Discussion.js
index 7bfb29b44..631f261d3 100644
--- a/framework/core/js/lib/models/Discussion.js
+++ b/framework/core/js/lib/models/Discussion.js
@@ -25,6 +25,7 @@ export default class Discussion extends mixin(Model, {
   readTime: Model.attribute('readTime', Model.transformDate),
   readNumber: Model.attribute('readNumber'),
   isUnread: computed('unreadCount', unreadCount => !!unreadCount),
+  isRead: computed('unreadCount', unreadCount => app.session.user && !unreadCount),
 
   canReply: Model.attribute('canReply'),
   canRename: Model.attribute('canRename'),
@@ -61,7 +62,7 @@ export default class Discussion extends mixin(Model, {
     const user = app.session.user;
 
     if (user && user.readTime() < this.lastTime()) {
-      return Math.max(0, this.lastPostNumber() - (this.readNumber() || 0))
+      return Math.max(0, this.lastPostNumber() - (this.readNumber() || 0));
     }
 
     return 0;
diff --git a/framework/core/less/forum/DiscussionListItem.less b/framework/core/less/forum/DiscussionListItem.less
index 2544edf80..da5228a29 100644
--- a/framework/core/less/forum/DiscussionListItem.less
+++ b/framework/core/less/forum/DiscussionListItem.less
@@ -44,8 +44,11 @@
   overflow: hidden;
   text-overflow: ellipsis;
 
+  .read & {
+    color: mix(@heading-color, @body-bg, 60%);
+  }
   .unread & {
-    font-weight: bold;
+    font-weight: 600;
   }
 }
 .DiscussionListItem-info {