double notification is not really something we need.

This commit is contained in:
Sam 2013-06-12 14:14:15 +10:00
parent 54d8c963d0
commit eca2931a63

View file

@ -75,7 +75,7 @@ Discourse = Ember.Application.createWithMixins({
$('title').text(title); $('title').text(title);
var notifyCount = this.get('notifyCount'); var notifyCount = this.get('notifyCount');
if (notifyCount > 0) { if (notifyCount > 0 && !Discourse.SiteSettings.dynamic_favicon) {
title = "(" + notifyCount + ") " + title; title = "(" + notifyCount + ") " + title;
} }
// chrome bug workaround see: http://stackoverflow.com/questions/2952384/changing-the-window-title-when-focussing-the-window-doesnt-work-in-chrome // chrome bug workaround see: http://stackoverflow.com/questions/2952384/changing-the-window-title-when-focussing-the-window-doesnt-work-in-chrome