mirror/php-flarum
1
0
Fork 0
mirror of https://github.com/flarum/core.git synced 2025-07-15 13:56:23 +02:00
Code Issues Releases Wiki Activity

Make unread discussion titles less overwhelming

Browse Source
This commit is contained in:
Toby Zerner
2015-09-08 10:27:50 +09:30
parent e8621636c5
commit 6a5427b600
3 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
js/forum/src/components/DiscussionListItem.js
View File

@ -51,6 +51,7 @@ export default class DiscussionListItem extends Component {
const discussion = this.props.discussion; const discussion = this.props.discussion;
const startUser = discussion.startUser(); const startUser = discussion.startUser();
const isUnread = discussion.isUnread(); const isUnread = discussion.isUnread();
const isRead = discussion.isRead();
const showUnread = !this.showRepliesCount() && isUnread; const showUnread = !this.showRepliesCount() && isUnread;
const jumpTo = Math.min(discussion.lastPostNumber(), (discussion.readNumber() || 0) + 1); const jumpTo = Math.min(discussion.lastPostNumber(), (discussion.readNumber() || 0) + 1);
const relevantPosts = this.props.params.q ? discussion.relevantPosts() : []; const relevantPosts = this.props.params.q ? discussion.relevantPosts() : [];
@ -71,7 +72,7 @@ export default class DiscussionListItem extends Component {
{icon('check')} {icon('check')}
</a> </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) : '#'} <a href={startUser ? app.route.user(startUser) : '#'}
className="DiscussionListItem-author" className="DiscussionListItem-author"
title={extractText(app.trans('core.discussion_started', {user: startUser, ago: humanTime(discussion.startTime())}))} title={extractText(app.trans('core.discussion_started', {user: startUser, ago: humanTime(discussion.startTime())}))}

3
js/lib/models/Discussion.js
View File

@ -25,6 +25,7 @@ export default class Discussion extends mixin(Model, {
readTime: Model.attribute('readTime', Model.transformDate), readTime: Model.attribute('readTime', Model.transformDate),
readNumber: Model.attribute('readNumber'), readNumber: Model.attribute('readNumber'),
isUnread: computed('unreadCount', unreadCount => !!unreadCount), isUnread: computed('unreadCount', unreadCount => !!unreadCount),
isRead: computed('unreadCount', unreadCount => app.session.user && !unreadCount),
canReply: Model.attribute('canReply'), canReply: Model.attribute('canReply'),
canRename: Model.attribute('canRename'), canRename: Model.attribute('canRename'),
@ -61,7 +62,7 @@ export default class Discussion extends mixin(Model, {
const user = app.session.user; const user = app.session.user;
if (user && user.readTime() < this.lastTime()) { 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; return 0;

5
less/forum/DiscussionListItem.less
View File

@ -44,8 +44,11 @@
overflow: hidden; overflow: hidden;
text-overflow: ellipsis; text-overflow: ellipsis;
.read & {
color: mix(@heading-color, @body-bg, 60%);
}
.unread & { .unread & {
font-weight: bold; font-weight: 600;
} }
} }
.DiscussionListItem-info { .DiscussionListItem-info {