Fix/nav upsell RTL padding (#37125)

* Fix navigation upsell RTL padding * changelog * Fix RTL notice close button * Try fix folded RTL * Try fix folded RTL * Try fix folded RTL * Try fix folded RTL * Try fix folded RTL * Update projects/packages/jetpack-mu-wpcom/src/features/wpcom-site-menu/wpcom-site-menu.php Co-authored-by: okmttdhr, tada <[email protected]> * Update changelog --------- Co-authored-by: okmttdhr, tada <[email protected]>
Automattic · May 1, 2024 · 3e9adbe · 3e9adbe
1 parent 8a1f680
commit 3e9adbe
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 3 deletions.
diff --git a/projects/packages/jetpack-mu-wpcom/changelog/fix-nav-upsell-rtl-padding b/projects/packages/jetpack-mu-wpcom/changelog/fix-nav-upsell-rtl-padding
@@ -0,0 +1,4 @@
+Significance: patch
+Type: changed
+
+Fix navigation upsell and notification RTL spacing.
diff --git a/projects/packages/jetpack-mu-wpcom/src/features/wpcom-site-menu/wpcom-site-menu.php b/projects/packages/jetpack-mu-wpcom/src/features/wpcom-site-menu/wpcom-site-menu.php
@@ -435,7 +435,7 @@ function wpcom_add_hosting_menu_intro_notice() {
 
 		.wpcom-site-menu-intro-notice a.close-button {
 			height: 16px;
-			margin-left: auto;
+			margin-inline-start: auto;
 		}
 	</style>
 	<div class="wpcom-site-menu-intro-notice notice notice-info" role="alert">

diff --git a/projects/packages/jetpack-mu-wpcom/src/features/wpcom-site-menu/wpcom-site-menu.scss b/projects/packages/jetpack-mu-wpcom/src/features/wpcom-site-menu/wpcom-site-menu.scss
@@ -59,8 +59,7 @@
 	}
 
 	.wp-menu-name {
-		padding-top: 0;
-		padding-left: 8px;
+		padding: 0 8px 8px 8px;
 	}
 
 	.upsell_banner {
@@ -148,6 +147,11 @@
 	@include folded;
 }
 
+.rtl.folded #adminmenu li.toplevel_page_site-notices .wp-menu-name {
+	right: auto;
+	padding-right: 4px;
+}
+
 @media only screen and (max-width: 960px) and (min-width: 783px) {
 	.auto-fold {
 		@include folded;