Skip to content

Commit

Permalink
Add a taxonomy survey notification ID to surveys.js
Browse files Browse the repository at this point in the history
If we're showing the new taxonomy navigation survey, we don't want to
show other banners.
  • Loading branch information
Mo Baig committed Mar 8, 2017
1 parent 9318488 commit bc5600b
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion app/assets/javascripts/surveys.js
Original file line number Diff line number Diff line change
Expand Up @@ -268,7 +268,13 @@
},

otherNotificationVisible: function() {
return $('#banner-notification:visible, #global-cookie-message:visible, #global-browser-prompt:visible').length > 0;
var notificationIds = [
'#banner-notification:visible',
'#global-cookie-message:visible',
'#global-browser-prompt:visible',
'#taxonomy-survey:visible'
]
return $(notificationIds.join(', ')).length > 0;
},

surveyTakenCookieName: function(survey) {
Expand Down

0 comments on commit bc5600b

Please sign in to comment.