From 851cb8b8f2b2806bf8bb51e3ca1b04ac7073e11e Mon Sep 17 00:00:00 2001
From: Jonathan Desrosiers <desrosj@git.wordpress.org>
Date: Wed, 15 May 2019 14:24:15 +0000
Subject: [PATCH] Site Health: Improve vertical alignment of icons in test
 result descriptions.

This change removes redundant CSS properties for different health check statuses in favor of utilizing the `.dashicons` class already in Core to define them more consistently. This also fixes the vertical alignment issues.

Props chetan200891, Clorith, desrosj.
Fixes #46940.

git-svn-id: https://develop.svn.wordpress.org/trunk@45309 602fd350-edb4-49c9-b593-d223f7449a82
---
 src/wp-admin/css/site-health.css               | 9 ---------
 src/wp-admin/includes/class-wp-site-health.php | 4 ++--
 2 files changed, 2 insertions(+), 11 deletions(-)

diff --git a/src/wp-admin/css/site-health.css b/src/wp-admin/css/site-health.css
index 93e90a6cfd..4294924bf9 100644
--- a/src/wp-admin/css/site-health.css
+++ b/src/wp-admin/css/site-health.css
@@ -153,32 +153,23 @@
 .health-check-body .pass::before,
 .health-check-body .good::before {
 	content: "\f147";
-	display: inline-block;
 	color: #46b450;
-	font-family: dashicons;
-	vertical-align: top;
 }
 
 .health-check-body .warning::before {
 	content: "\f460";
-	display: inline-block;
 	color: #ffb900;
-	font-family: dashicons;
 }
 
 .health-check-body .info::before {
 	content: "\f348";
-	display: inline-block;
 	color: #00a0d2;
-	font-family: dashicons;
 }
 
 .health-check-body .fail::before,
 .health-check-body .error::before {
 	content: "\f335";
-	display: inline-block;
 	color: #dc3232;
-	font-family: dashicons;
 }
 
 .site-health-copy-buttons {
diff --git a/src/wp-admin/includes/class-wp-site-health.php b/src/wp-admin/includes/class-wp-site-health.php
index a0137f798a..d7432eb371 100644
--- a/src/wp-admin/includes/class-wp-site-health.php
+++ b/src/wp-admin/includes/class-wp-site-health.php
@@ -889,7 +889,7 @@ class WP_Site_Health {
 					$result['status'] = 'recommended';
 				}
 
-				$failures[ $library ] = "<span class='$class'><span class='screen-reader-text'>$screen_reader</span></span> $message";
+				$failures[ $library ] = "<span class='dashicons $class'><span class='screen-reader-text'>$screen_reader</span></span> $message";
 			}
 		}
 
@@ -1487,7 +1487,7 @@ class WP_Site_Health {
 			}
 
 			$output .= sprintf(
-				'<li><span class="%s"><span class="screen-reader-text">%s</span></span> %s</li>',
+				'<li><span class="dashicons %s"><span class="screen-reader-text">%s</span></span> %s</li>',
 				esc_attr( $test->severity ),
 				$severity_string,
 				$test->description