From 447ca08b1f042035b4191b922eb9aac501e664e0 Mon Sep 17 00:00:00 2001
From: Chad Little <chad@chadsdomain.com>
Date: Wed, 11 Jun 2014 10:39:38 -0700
Subject: [PATCH] Add hover state to white header icons

Summary: Adds a hover state and color for white headers. Fixes T5318

Test Plan: Test on managing a dashboard.

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: epriestley, Korvin

Maniphest Tasks: T5318

Differential Revision: https://secure.phabricator.com/D9482
---
 resources/celerity/map.php                    |  4 +--
 .../layout/phabricator-action-header-view.css | 29 ++++++++++++++-----
 2 files changed, 24 insertions(+), 9 deletions(-)

diff --git a/resources/celerity/map.php b/resources/celerity/map.php
index d7c858ec60..1492b162e0 100644
--- a/resources/celerity/map.php
+++ b/resources/celerity/map.php
@@ -111,7 +111,7 @@ return array(
     'rsrc/css/font/font-awesome.css' => '73d075c3',
     'rsrc/css/font/font-source-sans-pro.css' => '91d53463',
     'rsrc/css/font/phui-font-icon-base.css' => 'eb84f033',
-    'rsrc/css/layout/phabricator-action-header-view.css' => '47bc4698',
+    'rsrc/css/layout/phabricator-action-header-view.css' => 'f11f18b5',
     'rsrc/css/layout/phabricator-action-list-view.css' => 'dcbfc854',
     'rsrc/css/layout/phabricator-crumbs-view.css' => '7fbf25b8',
     'rsrc/css/layout/phabricator-filetree-view.css' => 'fccf9f82',
@@ -685,7 +685,7 @@ return array(
     'paste-css' => 'aa1767d1',
     'path-typeahead' => 'f7fc67ec',
     'people-profile-css' => 'ba7b2762',
-    'phabricator-action-header-view-css' => '47bc4698',
+    'phabricator-action-header-view-css' => 'f11f18b5',
     'phabricator-action-list-view-css' => 'dcbfc854',
     'phabricator-application-launch-view-css' => '81bebcff',
     'phabricator-busy' => '6453c869',
diff --git a/webroot/rsrc/css/layout/phabricator-action-header-view.css b/webroot/rsrc/css/layout/phabricator-action-header-view.css
index bc153b8dc4..2ed1107311 100644
--- a/webroot/rsrc/css/layout/phabricator-action-header-view.css
+++ b/webroot/rsrc/css/layout/phabricator-action-header-view.css
@@ -69,18 +69,20 @@
   background: #fff;
 }
 
-.dashboard-panel .phabricator-action-header.gradient-white-header {
-  border: 1px solid {$lightblueborder};
-  border-bottom: 1px solid {$thinblueborder};
-  border-top-right-radius: 3px;
-  border-top-left-radius: 3px;
-}
-
 .phabricator-action-header.gradient-white-header .phabricator-action-header-title {
   color: {$darkbluetext};
   padding: 12px 0 12px 4px;
 }
 
+.phabricator-action-header.gradient-white-header
+ .phabricator-action-header-icon-list .phui-icon-view {
+  color: {$lightbluetext};
+}
+
+.phabricator-action-header.gradient-white-header
+ .phabricator-action-header-icon-list .phui-icon-view:hover {
+  color: {$darkbluetext};
+}
 
 .gradient-green-header .phabricator-action-header-title,
 .gradient-red-header .phabricator-action-header-title,
@@ -112,3 +114,16 @@
   border: 1px solid {$lightblueborder};
   border-bottom: 1px solid {$thinblueborder};
 }
+
+.dashboard-panel .phabricator-action-header.gradient-white-header {
+  border: 1px solid {$lightblueborder};
+  border-bottom: 1px solid {$thinblueborder};
+  border-top-right-radius: 3px;
+  border-top-left-radius: 3px;
+}
+
+.dashboard-panel .phabricator-action-header.gradient-white-header
+ .phabricator-action-header-icon-list {
+  padding-top: 6px;
+  padding-right: 6px;
+}