Skip to content

E
edx-platform
Commit cea0f925 by cahrens
Options

Make sure we don't mark the message bar as hiding if it is not shown.

parent 7f1c89b4
Showing with 6 additions and 4 deletions
cms/static/js/views/settings/advanced_view.js
...@@ -117,15 +117,17 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({ ...@@ -117,15 +117,17 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({
} }
}, },
showSaveCancelButtons: function(event) { showSaveCancelButtons: function(event) {
if (!this.buttonsVisible) { if (!this.notificationBarShowing) {
this.$el.find(".message-status").removeClass("is-shown"); this.$el.find(".message-status").removeClass("is-shown");
$('.wrapper-notification').removeClass('is-hiding').addClass('is-shown'); $('.wrapper-notification').removeClass('is-hiding').addClass('is-shown');
this.buttonsVisible = true; this.notificationBarShowing = true;
} }
}, },
hideSaveCancelButtons: function() { hideSaveCancelButtons: function() {
$('.wrapper-notification').removeClass('is-shown').addClass('is-hiding'); if (this.notificationBarShowing) {
this.buttonsVisible = false; $('.wrapper-notification').removeClass('is-shown').addClass('is-hiding');
this.notificationBarShowing = false;
}
}, },
saveView : function(event) { saveView : function(event) {
// TODO one last verification scan: // TODO one last verification scan:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment