From 71b7c80257867f6af745d829d8bcc2fe9caf2bdc Mon Sep 17 00:00:00 2001
From: riking <rikingcoding@gmail.com>
Date: Tue, 2 Sep 2014 13:29:28 -0700
Subject: [PATCH] UX: CSS tweaks for badge query previews

---
 .../admin/controllers/admin-badge-preview.js.es6     |  2 +-
 .../modal/admin_badge_preview.js.handlebars          |  5 ++---
 app/assets/stylesheets/common/admin/admin_base.scss  | 12 +++++++-----
 3 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/app/assets/javascripts/admin/controllers/admin-badge-preview.js.es6 b/app/assets/javascripts/admin/controllers/admin-badge-preview.js.es6
index e141dd24e40..70973e27813 100644
--- a/app/assets/javascripts/admin/controllers/admin-badge-preview.js.es6
+++ b/app/assets/javascripts/admin/controllers/admin-badge-preview.js.es6
@@ -19,7 +19,7 @@ export default Ember.Controller.extend({
 
   query_plan_html: function() {
     var raw = this.get('model.query_plan'),
-        returned = "<pre>";
+        returned = "<pre class='badge-query-plan'>";
 
     _.each(raw, function(linehash) {
       returned += Handlebars.Utils.escapeExpression(linehash["QUERY PLAN"]);
diff --git a/app/assets/javascripts/admin/templates/modal/admin_badge_preview.js.handlebars b/app/assets/javascripts/admin/templates/modal/admin_badge_preview.js.handlebars
index 12d78af55d1..d05abafdcaf 100644
--- a/app/assets/javascripts/admin/templates/modal/admin_badge_preview.js.handlebars
+++ b/app/assets/javascripts/admin/templates/modal/admin_badge_preview.js.handlebars
@@ -2,9 +2,8 @@
   {{#if errors}}
     <p class="error-header">{{i18n admin.badges.preview.sql_error_header}}</p>
 
-    <div class="badge-errors">
-      {{errors}}
-    </div>
+    <pre class="badge-errors">{{errors}}</pre>
+
     <!--
     TODO we want some help pages for this, link to those instead
     <p>
diff --git a/app/assets/stylesheets/common/admin/admin_base.scss b/app/assets/stylesheets/common/admin/admin_base.scss
index 748c21ff8a8..2c8f68b69bb 100644
--- a/app/assets/stylesheets/common/admin/admin_base.scss
+++ b/app/assets/stylesheets/common/admin/admin_base.scss
@@ -424,16 +424,18 @@ section.details {
     margin-left: 10px;
   }
 
-  .badge-query-plan, .badge-errors {
+  .badge-errors {
+    font-size: 14px;
+    line-height: 16px;
     padding: 4px;
     background-color: scale-color-diff();
   }
 
   .badge-query-plan {
-    font-size: 80%;
-  }
-  .badge-errors {
-    font-family: monospace;
+    font-size: 12px;
+    line-height: 13px;
+    padding: 4px;
+    background-color: scale-color-diff();
   }
 
   .count-warning {