Add tooltip component vertical-align as a sensible default

Automattic · Sep 21, 2023 · 6a55603 · 6a55603
1 parent 0b768b3
commit 6a55603
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/client/components/tooltip/style.scss b/client/components/tooltip/style.scss
@@ -17,6 +17,7 @@
 			transition: all 0.3s ease;
 			fill: currentColor;
 			margin: 0 0.4em;
+			vertical-align: text-bottom;
 
 			&:focus,
 			&:hover,

diff --git a/client/payment-details/dispute-details/style.scss b/client/payment-details/dispute-details/style.scss
@@ -81,12 +81,8 @@
 			border-bottom: 1px solid $wp-gray-5;
 		}
 
-		.wcpay-tooltip__content-wrapper.wcpay-tooltip--click__content-wrapper
-			> .wcpay-tooltip__content-wrapper
-			> [role='button'] {
-			margin-left: 0;
-			margin-right: 0;
-			vertical-align: text-bottom;
+		.wcpay-tooltip__content-wrapper > [role='button'] {
+			margin: 0;
 		}
 
 		&__response-date {