Sakura scheme: Fix color issues on masterbar (#37806)

Automattic · Jun 12, 2024 · 4e52d8f · 4e52d8f
1 parent d6e18b5
commit 4e52d8f
Show file tree

Hide file tree

Showing 6 changed files with 28 additions and 0 deletions.
diff --git a/projects/packages/masterbar/changelog/fix-sakura-scheme-issues-masterbar b/projects/packages/masterbar/changelog/fix-sakura-scheme-issues-masterbar
@@ -0,0 +1,4 @@
+Significance: patch
+Type: fixed
+
+Color Schemes: Fix Sakura color issues on masterbar
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
@@ -22,6 +22,7 @@ $menu-current-icon: $menu-current-text;
 $menu-current-background: $studio-blue-50; // Calypso --color-sidebar-menu-selected-background
 $menu-submenu-text: $studio-pink-0; // Calypso --color-sidebar-submenu-text
 $menu-submenu-background: $studio-pink-90; // Calypso --color-sidebar-submenu-background
+$menu-submenu-background-alt: $studio-pink-70; // Calypso --color-sidebar-submenu-background-alt
 $menu-submenu-focus-text: $studio-blue-20; // Calypso --color-sidebar-submenu-hover-text
 $menu-submenu-current-text: $studio-pink-0; // Calypso --color-sidebar-submenu-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/changelog/fix-sakura-scheme-issues-masterbar b/projects/plugins/jetpack/changelog/fix-sakura-scheme-issues-masterbar
@@ -0,0 +1,4 @@
+Significance: patch
+Type: other
+
+Color Schemes: Fix Sakura color issues on masterbar
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
@@ -22,6 +22,7 @@ $menu-current-icon: $menu-current-text;
 $menu-current-background: $studio-blue-50; // Calypso --color-sidebar-menu-selected-background
 $menu-submenu-text: $studio-pink-0; // Calypso --color-sidebar-submenu-text
 $menu-submenu-background: $studio-pink-90; // Calypso --color-sidebar-submenu-background
+$menu-submenu-background-alt: $studio-pink-70; // Calypso --color-sidebar-submenu-background-alt
 $menu-submenu-focus-text: $studio-blue-20; // Calypso --color-sidebar-submenu-hover-text
 $menu-submenu-current-text: $studio-pink-0; // Calypso --color-sidebar-submenu-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
@@ -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;
+}