Add some comments to Notifier.js

pull/21833/head
Richard van der Hoff 2016-03-29 10:51:13 +01:00
parent d7f8017b91
commit a12c1820df
1 changed files with 5 additions and 0 deletions

View File

@ -150,6 +150,8 @@ var Notifier = {
value: true value: true
}); });
}); });
// clear the notifications_hidden flag, so that if notifications are
// disabled again in the future, we will show the banner again.
this.setToolbarHidden(false); this.setToolbarHidden(false);
} else { } else {
if (!global.localStorage) return; if (!global.localStorage) return;
@ -188,6 +190,9 @@ var Notifier = {
setToolbarHidden: function(hidden, persistent = true) { setToolbarHidden: function(hidden, persistent = true) {
this.toolbarHidden = hidden; this.toolbarHidden = hidden;
// XXX: why are we dispatching this here?
// this is nothing to do with notifier_enabled
dis.dispatch({ dis.dispatch({
action: "notifier_enabled", action: "notifier_enabled",
value: this.isEnabled() value: this.isEnabled()