From 8396ad6430116c7a5735bf7d8fae6b3a34582cf1 Mon Sep 17 00:00:00 2001
From: Platonides <platonides@users.mediawiki.org>
Date: Mon, 2 Aug 2010 14:33:18 +0000
Subject: [PATCH] A <pre> and new lines is not the best element for the
 Selenium output. But at least do not use <font>

---
 maintenance/tests/selenium/SeleniumTestHTMLLogger.php | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/maintenance/tests/selenium/SeleniumTestHTMLLogger.php b/maintenance/tests/selenium/SeleniumTestHTMLLogger.php
index fb472174e3..21332cf000 100644
--- a/maintenance/tests/selenium/SeleniumTestHTMLLogger.php
+++ b/maintenance/tests/selenium/SeleniumTestHTMLLogger.php
@@ -3,7 +3,7 @@
 class SeleniumTestHTMLLogger {
 	public function setHeaders() {
 		global $wgOut;
-		$wgOut->addHeadItem( 'selenium', '<style>
+		$wgOut->addHeadItem( 'selenium', '<style type="text/css">
 		.selenium pre {
 			overflow-x: auto; /* Use horizontal scroller if needed; for Firefox 2, not needed in Firefox 3 */
 			white-space: pre-wrap; /* css-3 */
@@ -13,6 +13,7 @@ class SeleniumTestHTMLLogger {
 			/* width: 99%; */
 			word-wrap: break-word; /* Internet Explorer 5.5+ */
 		}
+		.selenium-success { color: green }
 		</style>' );
 	}
 
@@ -20,11 +21,11 @@ class SeleniumTestHTMLLogger {
 		global $wgOut;
 		$out = '';
 		if ( $mode == SeleniumTestSuite::RESULT_OK ) {
-			$out .= '<font color="green">';
+			$out .= '<span class="selenium-success">';
 		}
 		$out .= htmlspecialchars( $message );
 		if ( $mode == SeleniumTestSuite::RESULT_OK ) {
-			$out .= '</font>';
+			$out .= '</span>';
 		}
 		if ( $mode != SeleniumTestSuite::CONTINUE_LINE ) {
 			$out .= '<br />';
-- 
2.20.1