Changeset 50125

Timestamp:

02/01/2021 07:44:13 PM (3 years ago)

Author:

ryelle

Message:

Administration: Use correct color scheme color on admin bar icons when hovered.

This also removes a CSS rule in the light scheme which was flagged as temporary and overrides the hover text color.

Fixes #52229.

Location:

trunk/src/wp-admin/css/colors

Files:

• _admin.scss (modified) (2 diffs)
• light/colors.scss (modified) (1 diff)

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

@@ -340 +340 @@
 #adminmenu li.wp-has-current-submenu div.wp-menu-image:before,
 #adminmenu a.current:hover div.wp-menu-image:before,
+#adminmenu li.current div.wp-menu-image:before,
 #adminmenu li.wp-has-current-submenu a:focus div.wp-menu-image:before,
 #adminmenu li.wp-has-current-submenu.opensub div.wp-menu-image:before,
@@ -417 +418 @@
 #wpadminbar:not(.mobile) li:hover .ab-item:after,
 #wpadminbar:not(.mobile) li:hover #adminbarsearch:before {
-    color: $menu-highlight-icon;
+    color: $menu-submenu-focus-text;
 }
 

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

@@ -20 +20 @@
 @import "../_admin.scss";
 
-/* temporary fix for admin-bar hover color */
-#wpadminbar .ab-top-menu > li:hover > .ab-item,
-#wpadminbar .ab-top-menu > li.hover > .ab-item,
-#wpadminbar > #wp-toolbar > #wp-admin-bar-root-default li:hover span.ab-label,
-#wpadminbar > #wp-toolbar > #wp-admin-bar-top-secondary li.hover span.ab-label,
-#wpadminbar .ab-top-menu > li > .ab-item:focus,
-#wpadminbar.nojq .quicklinks .ab-top-menu > li > .ab-item:focus,
-#wpadminbar.nojs .ab-top-menu > li.menupop:hover > .ab-item,
-#wpadminbar .ab-top-menu > li.menupop.hover > .ab-item {
-    color: $text-color;
-}
-
 /* Override the theme filter highlight color for this scheme */
 .theme-section.current,