github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-23 23:54:16 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Merge pull request #4541 from xfalcox/no-last-visit-on-top

Browse Source 
FIX: Last Visit line shouldn't appear on /top
This commit is contained in:
Jeff Atwood 2016-11-08 23:08:40 +01:00 committed by GitHub
commit 13a40d2f99
2 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
app/assets/javascripts/discourse/components/topic-list.js.es6
View File

@ -40,12 +40,12 @@ export default Ember.Component.extend({
}
},
@observes('topics', 'order', 'ascending', 'category')
@observes('topics', 'order', 'ascending', 'category', 'top')
lastVisitedTopicChanged() {
this.refreshLastVisited();
},
_updateLastVisitedTopic(topics, order, ascending) {
_updateLastVisitedTopic(topics, order, ascending, top) {
this.set('lastVisitedTopic', null);
@ -57,6 +57,10 @@ export default Ember.Component.extend({
return;
}
if (top) {
return;
}
if (!topics || topics.length === 1) {
return;
}
@ -103,7 +107,7 @@ export default Ember.Component.extend({
},
refreshLastVisited() {
this._updateLastVisitedTopic(this.get('topics'), this.get('order'), this.get('ascending'));
this._updateLastVisitedTopic(this.get('topics'), this.get('order'), this.get('ascending'), this.get('top'));
},
click(e) {

1
app/assets/javascripts/discourse/templates/discovery/topics.hbs
View File

@ -34,6 +34,7 @@
{{#if hasTopics}}
{{topic-list
highlightLastVisited=true
top=top
showTopicPostBadges=showTopicPostBadges
showPosters=true
currentUser=currentUser