From a12c1820dfe64d1f742213376b0adbfdc32ced3d Mon Sep 17 00:00:00 2001 From: Richard van der Hoff Date: Tue, 29 Mar 2016 10:51:13 +0100 Subject: [PATCH] Add some comments to Notifier.js --- src/Notifier.js | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/Notifier.js b/src/Notifier.js index 827d173915..99fef9d671 100644 --- a/src/Notifier.js +++ b/src/Notifier.js @@ -150,6 +150,8 @@ var Notifier = { 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); } else { if (!global.localStorage) return; @@ -188,6 +190,9 @@ var Notifier = { setToolbarHidden: function(hidden, persistent = true) { this.toolbarHidden = hidden; + + // XXX: why are we dispatching this here? + // this is nothing to do with notifier_enabled dis.dispatch({ action: "notifier_enabled", value: this.isEnabled()