Changeset 29287

Timestamp:

07/24/2014 09:06:42 PM (12 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 {