fix: error message display in production

only use the <pre> tag if the response was a text message i.e. during dev errors.
Use the alert-danger class of bootstrap 3 rather than alert-error.

2 files changed, 2 insertions, 2 deletions

diff --git a/app/assets/javascripts/leap.js b/app/assets/javascripts/leap.js
index c12a2d6..5084efa 100644
--- a/app/assets/javascripts/leap.js
+++ b/app/assets/javascripts/leap.js
@@ -3,7 +3,7 @@
 // add a bootstrap alert to the page via javascript.
 //
 function alert_message(msg) {
-  $('#messages').append('<div class="alert alert-error"><a class="close" data-dismiss="alert">×</a><pre>'+msg+'</pre></div>');
+  $('#messages').append('<div class="alert alert-danger"><a class="close" data-dismiss="alert">×</a>'+msg+'</div>');
 }
 
 ClientSideValidations.formBuilders['SimpleForm::FormBuilder'].wrappers.bootstrap = ClientSideValidations.formBuilders['SimpleForm::FormBuilder'].wrappers["default"];
diff --git a/app/assets/javascripts/users.js b/app/assets/javascripts/users.js
index 9a10532..d080b49 100644
--- a/app/assets/javascripts/users.js
+++ b/app/assets/javascripts/users.js
@@ -139,7 +139,7 @@
       return message.errors || { base: message.error };
     } else {
       return {
-        base: message
+        base: '<pre>' + message + '</pre>'
       };
     }
   }