From: Platonides <platonides@users.mediawiki.org>
Date: Mon, 2 Aug 2010 14:33:18 +0000 (+0000)
Subject: A <pre> and new lines is not the best element for the Selenium output.
X-Git-Tag: 1.31.0-rc.0~35778
X-Git-Url: http://git.cyclocoop.org/%40spipnet%40?a=commitdiff_plain;h=8396ad6430116c7a5735bf7d8fae6b3a34582cf1;p=lhc%2Fweb%2Fwiklou.git

A <pre> and new lines is not the best element for the Selenium output.
But at least do not use <font>
---

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 />';