Fix menu hover color

Automattic · Jun 12, 2024 · c198264 · c198264
1 parent 7512dd1
commit c198264
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 2 deletions.
diff --git a/projects/packages/masterbar/src/admin-color-schemes/colors/sakura/_variables.scss b/projects/packages/masterbar/src/admin-color-schemes/colors/sakura/_variables.scss
@@ -14,7 +14,7 @@ $body-background: $studio-gray-0; // Calypso --color-surface-backdrop
 // admin menu & admin-bar (core _variables.scss)
 $menu-text: $studio-pink-80; // Calypso --color-sidebar-text
 $menu-background: $studio-pink-5; // Calypso --color-sidebar-background
-$menu-highlight-text: $studio-pink-0; // Calypso --color-sidebar-menu-hover-text
+$menu-highlight-text: $studio-pink-90; // Calypso --color-sidebar-menu-hover-text
 $menu-highlight-icon: $menu-highlight-text;
 $menu-highlight-background: $studio-pink-10; // Calypso --color-sidebar-menu-hover-background
 $menu-current-text: $studio-white; // Calypso --color-sidebar-menu-selected-text

diff --git a/projects/packages/masterbar/src/admin-color-schemes/colors/sakura/colors.scss b/projects/packages/masterbar/src/admin-color-schemes/colors/sakura/colors.scss
@@ -1,3 +1,12 @@
 @import "variables";
 @import "../admin";
 @import "sidebar-notice";
+
+#wpadminbar:not(.mobile) li:hover .ab-icon:before,
+#wpadminbar:not(.mobile) li:hover .ab-item:before {
+    color: $menu-submenu-current-text;
+}
+
+#wpadminbar #wp-admin-bar-user-info .display-name {
+	color: $menu-submenu-text;
+}
diff --git a/projects/plugins/jetpack/modules/masterbar/admin-color-schemes/colors/sakura/_variables.scss b/projects/plugins/jetpack/modules/masterbar/admin-color-schemes/colors/sakura/_variables.scss
@@ -14,7 +14,7 @@ $body-background: $studio-gray-0; // Calypso --color-surface-backdrop
 // admin menu & admin-bar (core _variables.scss)
 $menu-text: $studio-pink-80; // Calypso --color-sidebar-text
 $menu-background: $studio-pink-5; // Calypso --color-sidebar-background
-$menu-highlight-text: $studio-pink-0; // Calypso --color-sidebar-menu-hover-text
+$menu-highlight-text: $studio-pink-90; // Calypso --color-sidebar-menu-hover-text
 $menu-highlight-icon: $menu-highlight-text;
 $menu-highlight-background: $studio-pink-10; // Calypso --color-sidebar-menu-hover-background
 $menu-current-text: $studio-white; // Calypso --color-sidebar-menu-selected-text

diff --git a/projects/plugins/jetpack/modules/masterbar/admin-color-schemes/colors/sakura/colors.scss b/projects/plugins/jetpack/modules/masterbar/admin-color-schemes/colors/sakura/colors.scss
@@ -2,6 +2,11 @@
 @import "../admin";
 @import "sidebar-notice";
 
+#wpadminbar:not(.mobile) li:hover .ab-icon:before,
+#wpadminbar:not(.mobile) li:hover .ab-item:before {
+    color: $menu-submenu-current-text;
+}
+
 #wpadminbar #wp-admin-bar-user-info .display-name {
 	color: $menu-submenu-text;
 }