From d7b9b8b2ce2987ebcadc4d36f404db5243d7c1d1 Mon Sep 17 00:00:00 2001
From: Chad Little <chad@chadsdomain.com>
Date: Tue, 3 Feb 2015 09:07:00 -0800
Subject: [PATCH] ConfigIssueView minor tweaks

Summary: Minor, adds border, reduces greys, etc.

Test Plan:
View a number of config issues, see new colors.

{F282035}

Reviewers: btrahan, epriestley

Reviewed By: epriestley

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D11650
---
 resources/celerity/map.php                    |  4 +--
 .../PhabricatorConfigIssueViewController.php  |  1 +
 .../css/application/config/setup-issue.css    | 31 +++++++++----------
 3 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/resources/celerity/map.php b/resources/celerity/map.php
index 3a79325338..5a92509bba 100644
--- a/resources/celerity/map.php
+++ b/resources/celerity/map.php
@@ -43,7 +43,7 @@ return array(
     'rsrc/css/application/config/config-options.css' => '7fedf08b',
     'rsrc/css/application/config/config-template.css' => '25d446d6',
     'rsrc/css/application/config/config-welcome.css' => 'b0d16200',
-    'rsrc/css/application/config/setup-issue.css' => '8f852bc0',
+    'rsrc/css/application/config/setup-issue.css' => '22270af2',
     'rsrc/css/application/config/unhandled-exception.css' => '37d4f9a2',
     'rsrc/css/application/conpherence/menu.css' => '73774137',
     'rsrc/css/application/conpherence/message-pane.css' => '17a9517f',
@@ -814,7 +814,7 @@ return array(
     'releeph-preview-branch' => 'b7a6f4a5',
     'releeph-request-differential-create-dialog' => '8d8b92cd',
     'releeph-request-typeahead-css' => '667a48ae',
-    'setup-issue-css' => '8f852bc0',
+    'setup-issue-css' => '22270af2',
     'sprite-gradient-css' => '4bdb98a7',
     'sprite-login-css' => 'a355d921',
     'sprite-main-header-css' => '92720ee2',
diff --git a/src/applications/config/controller/PhabricatorConfigIssueViewController.php b/src/applications/config/controller/PhabricatorConfigIssueViewController.php
index 6c169e82a9..dc0b68a452 100644
--- a/src/applications/config/controller/PhabricatorConfigIssueViewController.php
+++ b/src/applications/config/controller/PhabricatorConfigIssueViewController.php
@@ -38,6 +38,7 @@ final class PhabricatorConfigIssueViewController
 
     $crumbs = $this
       ->buildApplicationCrumbs()
+      ->setBorder(true)
       ->addTextCrumb(pht('Setup Issues'), $this->getApplicationURI('issue/'))
       ->addTextCrumb($title, $request->getRequestURI());
 
diff --git a/webroot/rsrc/css/application/config/setup-issue.css b/webroot/rsrc/css/application/config/setup-issue.css
index f237e3c8b9..8a5d2333e6 100644
--- a/webroot/rsrc/css/application/config/setup-issue.css
+++ b/webroot/rsrc/css/application/config/setup-issue.css
@@ -23,36 +23,36 @@
 }
 
 .setup-issue table {
-  width: 90%;
-  margin: auto;
+  width: 100%;
   border-collapse: collapse;
-  border: 1px solid #dfdfdf;
+  border: 1px solid #BFCFDA;
 }
 
 .setup-issue table th {
   text-align: right;
   width: 30%;
-  background: #efefef;
-  border: 1px solid #dfdfdf;
+  background: #F8F9FC;
+  border: 1px solid #BFCFDA;
   padding: 8px;
 }
 
 .setup-issue table td {
-  border: 1px solid #dfdfdf;
+  border: 1px solid #BFCFDA;
   padding: 8px;
 }
 
 .setup-issue pre {
-  width: 95%;
-  margin: auto;
-  border: 1px solid #dfdfdf;
+  width: auto;
+  border: 1px solid #BFCFDA;
   padding: 10px 2%;
-  background: #efefef;
+  background: #F8F9FC;
   overflow-x: auto;
 }
 
 .setup-issue tt {
-  color: #74777D;
+  color: black;
+  background: #EBECEE;
+  padding: 2px 4px;
 }
 
 .setup-issue em {
@@ -77,7 +77,7 @@
 }
 
 .setup-issue-name {
-  color: #6B748C;
+  color: #464C5C;
   text-shadow: 0 1px #fff;
   padding: 16px 12px;
   border-bottom: 1px solid #BFCFDA;
@@ -93,10 +93,9 @@
 
 .setup-issue-status {
   padding: 12px;
-  background: #f3f3f3;
-  color: #666666;
-  border-bottom: 1px solid #dfdfdf;
-  font-style: italic;
+  background: #FDF5D4;
+  color: #bc7837;
+  border-bottom: 1px solid #BFCFDA;
 }
 
 .setup-issue-actions {