From 1b01b92dc6c2ec523d58297cba87ba5b487bd3e2 Mon Sep 17 00:00:00 2001
From: Scott Taylor <wonderboymusic@git.wordpress.org>
Date: Thu, 24 Jul 2014 21:06:42 +0000
Subject: [PATCH] Media Grid CSS: Media modal close icon should be consistent
 with other modals.

Props celloexpressions.
Fixes #29012.


git-svn-id: https://develop.svn.wordpress.org/trunk@29287 602fd350-edb4-49c9-b593-d223f7449a82
---
 src/wp-admin/css/colors/_admin.scss |  9 +++++
 src/wp-includes/css/media-views.css | 55 +++++++++++++----------------
 2 files changed, 33 insertions(+), 31 deletions(-)

diff --git a/src/wp-admin/css/colors/_admin.scss b/src/wp-admin/css/colors/_admin.scss
index a08296f847..9c560ce7e2 100644
--- a/src/wp-admin/css/colors/_admin.scss
+++ b/src/wp-admin/css/colors/_admin.scss
@@ -416,6 +416,15 @@ ul#adminmenu > li.current > a.current:after {
 	box-shadow: 0 0 0 1px #fff, 0 0 0 2px $highlight-color;
 }
 
+.media-modal-close:hover,
+.edit-attachment-frame .edit-media-header .right:hover,
+.edit-attachment-frame .edit-media-header .left:hover,
+.media-modal-close:focus,
+.edit-attachment-frame .edit-media-header .right:focus,
+.edit-attachment-frame .edit-media-header .left:focus {
+	background: $menu-highlight-background;
+	color: $menu-highlight-text;
+}
 
 /* Themes */
 
diff --git a/src/wp-includes/css/media-views.css b/src/wp-includes/css/media-views.css
index a75bddc714..adbb794cf3 100644
--- a/src/wp-includes/css/media-views.css
+++ b/src/wp-includes/css/media-views.css
@@ -141,33 +141,40 @@
 .media-modal-close {
 	position: absolute;
 	text-decoration: none;
-	top: 5px;
-	right: 10px;
-	width: 30px;
-	height: 30px;
+	top: 0;
+	right: 0;
+	width: 50px;
+	height: 48px;
 	z-index: 1000;
+	color: #777;
 }
 
 .media-modal-close span.media-modal-icon {
-	display: block;
+	display: none;
 	margin: 8px auto 0;
 	width: 15px;
 	height: 15px;
 	background-image: none;
 }
 
-.media-modal-close .media-modal-icon:before {
-	content: '\f158';
-	font: normal 20px/1 'dashicons';
+.media-modal-close:before {
+	position: absolute;
+	top: 1px;
+	left: 11px;
+	font: normal 30px/50px 'dashicons' !important;
+	content: '\f335';
+	font-weight: 300;
 	speak: none;
-	vertical-align: middle;
 	-webkit-font-smoothing: antialiased;
 	-moz-osx-font-smoothing: grayscale;
-	color: #666;
 }
 
-.media-modal-close:hover .media-modal-icon:before {
-	color: #2ea2cc;
+.media-modal-close:focus,
+.media-modal-close:hover {
+	background: #0074a2;
+	color: #fff;
+	-webkit-box-shadow: none;
+	box-shadow: none;
 }
 
 .media-modal-close:active {
@@ -2688,20 +2695,9 @@ video#inline-media-node {
 	overflow: hidden;
 }
 
-.upload-php .media-modal-close .media-modal-icon:before {
-	content: '\f335';
-	font-size: 30px;
-}
-
-.upload-php .media-modal-close:hover .media-modal-icon:before,
-.upload-php .media-modal-close:focus .media-modal-icon:before {
-	color: #fff;
-}
-
-.upload-php .media-modal-close .media-modal-icon {
-	margin-top: 13px;
-	width: 30px;
-	height: 30px;
+.upload-php .media-modal-close:before {
+	top: 4px;
+	left: 14px;
 }
 
 .upload-php .media-modal-close,
@@ -2719,11 +2715,6 @@ video#inline-media-node {
 	border-left: 1px solid #ddd;
 }
 
-.upload-php .media-modal-close {
-	top: 0;
-	right: 0;
-}
-
 .edit-attachment-frame .edit-media-header .left {
 	right: 112px;
 }
@@ -2760,6 +2751,8 @@ video#inline-media-node {
 
 .edit-attachment-frame .edit-media-header .left.disabled,
 .edit-attachment-frame .edit-media-header .right.disabled,
+.edit-attachment-frame .edit-media-header .left.disabled:focus,
+.edit-attachment-frame .edit-media-header .right.disabled:focus,
 .edit-attachment-frame .edit-media-header .left.disabled:hover,
 .edit-attachment-frame .edit-media-header .right.disabled:hover {
 	color: #ccc;