move nag bar clear statement to any desktop notif toggle not just 0->1
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>pull/21833/head
parent
41dee71d38
commit
0a638cd545
|
@ -170,15 +170,15 @@ const 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(true);
|
|
||||||
} else {
|
} else {
|
||||||
dis.dispatch({
|
dis.dispatch({
|
||||||
action: "notifier_enabled",
|
action: "notifier_enabled",
|
||||||
value: false,
|
value: false,
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
// set the notifications_hidden flag, as the user has knowingly interacted
|
||||||
|
// with the setting we shouldn't nag them any further
|
||||||
|
this.setToolbarHidden(true);
|
||||||
},
|
},
|
||||||
|
|
||||||
isEnabled: function() {
|
isEnabled: function() {
|
||||||
|
|
Loading…
Reference in New Issue