Skip to content

Commit

Permalink
Merge pull request #900 from adaptlearning/issue/899
Browse files Browse the repository at this point in the history
Fix issue with alerts showing confirm buttons
  • Loading branch information
brian-learningpool committed Oct 21, 2015
2 parents 749ce7f + 50cb207 commit 77ad571
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions frontend/src/core/notify/plugins/alert/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,16 +19,17 @@ define(function(require) {
defaults.confirmButtonText = "Yes, I'm sure";
defaults.cancelButtonText = "No";
break;
case "input":
case "success":
case "info":
case "error":
case "warning":
defaults.showCancelButton = true;
defaults.confirmButtonText = "Yes, I'm sure";
defaults.cancelButtonText = "No";
defaults.confirmButtonColor = "#DD6B55";
break;
case "input":
case "success":
case "info":
case "error":
break;
default:
if(data.type) {
Origin. Notify.console({
Expand Down Expand Up @@ -64,7 +65,7 @@ define(function(require) {
text: data
};
}

data.type = data.type || "confirm";
openPopup(data);
};
Expand Down

0 comments on commit 77ad571

Please sign in to comment.