Discource-C
/
discourse
mirror of https://github.com/discourse/discourse.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

FIX: never ask for less than 5 notifications

This commit is contained in:
Sam 2015-09-04 17:22:26 +10:00
parent 1f31435a7b
commit b3d6cefef4
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
app/assets/javascripts/discourse/components/user-menu.js.es6
View File

@ -46,6 +46,9 @@ export default Ember.Component.extend({
// estimate (poorly) the amount of notifications to return // estimate (poorly) the amount of notifications to return
const limit = Math.round(($(window).height() - headerHeight()) / 50); const limit = Math.round(($(window).height() - headerHeight()) / 50);
// we REALLY don't want to be asking for negative counts of notifications
// less than 5 is also not that useful
if (limit < 5) { limit = 5; }
// TODO: It's a bit odd to use the store in a component, but this one really // TODO: It's a bit odd to use the store in a component, but this one really
// wants to reach out and grab notifications // wants to reach out and grab notifications