Skip to content

A
atlas
Commit ad226ddd by kevalbhatt
Options

ATLAS-3711 : UI: Classification/Term modal should close after…

ATLAS-3711 : UI: Classification/Term modal should close after create/update/delete response (#3 PropagationPropertyModal loader added)
parent 490e37ec
Showing with 4 additions and 8 deletions
dashboardv2/public/js/views/graph/PropagationPropertyModal.js
...@@ -236,9 +236,6 @@ define(['require', ...@@ -236,9 +236,6 @@ define(['require',
entityId = that.ui.propagationOptions.attr('entity-id'), entityId = that.ui.propagationOptions.attr('entity-id'),
PropagationValue = this.$("input[name='propagateRelation']:checked").val(), PropagationValue = this.$("input[name='propagateRelation']:checked").val(),
relationshipProp = {}; relationshipProp = {};
if (PropagationValue === this.ui.propagationOptions.attr("propagation")) {
return;
}
this.ui.propagationOptions.attr("propagation", PropagationValue); this.ui.propagationOptions.attr("propagation", PropagationValue);
if (this.viewType == "flow") { if (this.viewType == "flow") {
relationshipProp = { relationshipProp = {
...@@ -306,11 +303,12 @@ define(['require', ...@@ -306,11 +303,12 @@ define(['require',
this.ui.PropagatedClassificationTable.append(_.isEmpty(propagationStringValue) ? "No Records Found." : classificationTableValue); this.ui.PropagatedClassificationTable.append(_.isEmpty(propagationStringValue) ? "No Records Found." : classificationTableValue);
}, },
showLoader: function() { showLoader: function() {
this.modal.$el.find('button.ok').attr("disabled", true); this.modal.$el.find('button.ok').showButtonLoader();
this.$('.overlay').removeClass('hide').addClass('show'); this.$('.overlay').removeClass('hide').addClass('show');
}, },
hideLoader: function(options) { hideLoader: function(options) {
var buttonDisabled = options && options.buttonDisabled; var buttonDisabled = options && options.buttonDisabled;
this.modal.$el.find('button.ok').hideButtonLoader();
this.modal.$el.find('button.ok').attr("disabled", buttonDisabled ? buttonDisabled : false); this.modal.$el.find('button.ok').attr("disabled", buttonDisabled ? buttonDisabled : false);
this.$('.overlay').removeClass('show').addClass('hide'); this.$('.overlay').removeClass('show').addClass('hide');
}, },
......
dashboardv3/public/js/views/graph/PropagationPropertyModal.js
...@@ -236,9 +236,6 @@ define(['require', ...@@ -236,9 +236,6 @@ define(['require',
entityId = that.ui.propagationOptions.attr('entity-id'), entityId = that.ui.propagationOptions.attr('entity-id'),
PropagationValue = this.$("input[name='propagateRelation']:checked").val(), PropagationValue = this.$("input[name='propagateRelation']:checked").val(),
relationshipProp = {}; relationshipProp = {};
if (PropagationValue === this.ui.propagationOptions.attr("propagation")) {
return;
}
this.ui.propagationOptions.attr("propagation", PropagationValue); this.ui.propagationOptions.attr("propagation", PropagationValue);
if (this.viewType == "flow") { if (this.viewType == "flow") {
relationshipProp = { relationshipProp = {
...@@ -306,11 +303,12 @@ define(['require', ...@@ -306,11 +303,12 @@ define(['require',
this.ui.PropagatedClassificationTable.append(_.isEmpty(propagationStringValue) ? "No Records Found." : classificationTableValue); this.ui.PropagatedClassificationTable.append(_.isEmpty(propagationStringValue) ? "No Records Found." : classificationTableValue);
}, },
showLoader: function() { showLoader: function() {
this.modal.$el.find('button.ok').attr("disabled", true); this.modal.$el.find('button.ok').showButtonLoader();
this.$('.overlay').removeClass('hide').addClass('show'); this.$('.overlay').removeClass('hide').addClass('show');
}, },
hideLoader: function(options) { hideLoader: function(options) {
var buttonDisabled = options && options.buttonDisabled; var buttonDisabled = options && options.buttonDisabled;
this.modal.$el.find('button.ok').hideButtonLoader();
this.modal.$el.find('button.ok').attr("disabled", buttonDisabled ? buttonDisabled : false); this.modal.$el.find('button.ok').attr("disabled", buttonDisabled ? buttonDisabled : false);
this.$('.overlay').removeClass('show').addClass('hide'); this.$('.overlay').removeClass('show').addClass('hide');
}, },
......
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