Commit 75ceed7f by cahrens

Show "successful save" message when a new edit is made.

parent e9bb5eb1
...@@ -4,7 +4,6 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({ ...@@ -4,7 +4,6 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({
// the key for a newly added policy-- before the user has entered a key value // the key for a newly added policy-- before the user has entered a key value
new_key : "__new_advanced_key__", new_key : "__new_advanced_key__",
error_saving : "error_saving", error_saving : "error_saving",
unsaved_changes: "unsaved_changes",
successful_changes: "successful_changes", successful_changes: "successful_changes",
// Model class is CMS.Models.Settings.Advanced // Model class is CMS.Models.Settings.Advanced
...@@ -67,7 +66,6 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({ ...@@ -67,7 +66,6 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({
// TODO: error checking // TODO: error checking
var JSONValue = JSON.parse(quotedValue); var JSONValue = JSON.parse(quotedValue);
self.model.set(key, JSONValue, {validate: true}); self.model.set(key, JSONValue, {validate: true});
self.showMessage(self.unsaved_changes);
} }
}); });
}, },
...@@ -78,9 +76,6 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({ ...@@ -78,9 +76,6 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({
if (type === this.error_saving) { if (type === this.error_saving) {
this.$el.find(".message-status.error").addClass("is-shown"); this.$el.find(".message-status.error").addClass("is-shown");
} }
else if (type === this.unsaved_changes) {
$('.wrapper-notification').addClass('is-shown');
}
else if (type === this.successful_changes) { else if (type === this.successful_changes) {
this.$el.find(".message-status.confirm").addClass("is-shown"); this.$el.find(".message-status.confirm").addClass("is-shown");
this.hideSaveCancelButtons(); this.hideSaveCancelButtons();
...@@ -95,6 +90,7 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({ ...@@ -95,6 +90,7 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({
showSaveCancelButtons: function() { showSaveCancelButtons: function() {
if (!this.buttonsVisible) { if (!this.buttonsVisible) {
this.$el.find(".message-status").removeClass("is-shown");
$('.wrapper-notification').addClass('is-shown'); $('.wrapper-notification').addClass('is-shown');
this.buttonsVisible = true; this.buttonsVisible = true;
} }
...@@ -128,7 +124,7 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({ ...@@ -128,7 +124,7 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({
this.model.unset(key); this.model.unset(key);
} }
li$.remove(); li$.remove();
this.showMessage(this.unsaved_changes); this.showSaveCancelButtons();
}, },
saveView : function(event) { saveView : function(event) {
// TODO one last verification scan: // TODO one last verification scan:
...@@ -165,7 +161,7 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({ ...@@ -165,7 +161,7 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({
// only 1 but hey, let's take advantage of the context mechanism // only 1 but hey, let's take advantage of the context mechanism
_.each(policyValueDivs, this.attachJSONEditor, this); _.each(policyValueDivs, this.attachJSONEditor, this);
this.toggleNewButton(false); this.toggleNewButton(false);
this.showMessage(this.unsaved_changes); this.showSaveCancelButtons();
}, },
updateKey : function(event) { updateKey : function(event) {
// old key: either the key as in the model or new_key. // old key: either the key as in the model or new_key.
...@@ -233,7 +229,6 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({ ...@@ -233,7 +229,6 @@ CMS.Views.Settings.Advanced = CMS.Views.ValidatingView.extend({
_.each(policyValueDivs, this.attachJSONEditor, this); _.each(policyValueDivs, this.attachJSONEditor, this);
this.fieldToSelectorMap[newKey] = newKey; this.fieldToSelectorMap[newKey] = newKey;
this.showMessage(this.unsaved_changes);
} }
}, },
validateKey : function(oldKey, newKey) { validateKey : function(oldKey, newKey) {
......
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