Changeset 29287

Timestamp:

07/24/2014 09:06:42 PM (11 years ago)

Author:

wonderboymusic

Message:

Media Grid CSS: Media modal close icon should be consistent with other modals.

Props celloexpressions.
Fixes #29012.

Location:

trunk/src

Files:

• wp-admin/css/colors/_admin.scss (modified) (1 diff)
• wp-includes/css/media-views.css (modified) (5 diffs)

trunk/src/wp-admin/css/colors/_admin.scss

@@ -417 +417 @@
 }
 
+.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 */

trunk/src/wp-includes/css/media-views.css

@@ -142 +142 @@
     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;
@@ -157 +158 @@
 }
 
-.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;
 }
 
@@ -2689 +2696 @@
 }
 
-.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;
 }
 
@@ -2720 +2716 @@
 }
 
-.upload-php .media-modal-close {
-    top: 0;
-    right: 0;
-}
-
 .edit-attachment-frame .edit-media-header .left {
     right: 112px;
@@ -2761 +2752 @@
 .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 {