Skip to content

Commit

Permalink
Quick fix to make sure that the alert is displayed for all users. Wit…
Browse files Browse the repository at this point in the history
…hout this fix, users who closed the previous alert would not have seen this alert
  • Loading branch information
mortnod committed Jan 29, 2015
1 parent 6f793cf commit 9338757
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions js/alerts.js
Expand Up @@ -20,11 +20,11 @@ Alerts = {
},

shouldAlertBeDisplayed: function() {
if (localStorage['alert_closed'] === undefined) {
if (localStorage['office365_alert_closed'] === undefined) {
return true;
}
else {
return !JSON.parse(localStorage['alert_closed']);
return !JSON.parse(localStorage['office365_alert_closed']);
}
},

Expand All @@ -36,7 +36,7 @@ Alerts = {

bindCloseButton: function() {
$('.alert .close').click(function(){
localStorage['alert_closed'] = 'true';
localStorage['office365_alert_closed'] = 'true';
$('.alert').fadeOut(function() {
$('#grid').removeClass('alert-padding');
});
Expand Down

0 comments on commit 9338757

Please sign in to comment.