From: Brion Vibber <brion@users.mediawiki.org>
Date: Mon, 30 Jan 2006 03:34:23 +0000 (+0000)
Subject: * (bug 1284) Inline styles for diffs in Recent Changes RSS/Atom feeds
X-Git-Tag: 1.6.0~398
X-Git-Url: http://git.cyclocoop.org/%22%20.%20generer_url_ecrire%28%22sites_tous%22%29%20.%20%22?a=commitdiff_plain;h=ac57f442505533bd0d159e2c716c73d69c6041c9;p=lhc%2Fweb%2Fwiklou.git

* (bug 1284) Inline styles for diffs in Recent Changes RSS/Atom feeds
In a perfect world, HTML would have a version of the <style> element
which applies only to its own contents and is allowed in the body.
Or, secondarily, XSLT could be much much easier to use.
---

diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index 0136bb9b78..ce3e2faa4c 100644
--- a/RELEASE-NOTES
+++ b/RELEASE-NOTES
@@ -584,6 +584,7 @@ fully support the editing toolbar, but was found to be too confusing.
   Edit messages like "MediaWiki:Namespacenotice-" plus namespace name
   which is blank for main namespace, or like e.g. "User_talk"
 * Adjust user login/creation form hooks to work with a captcha plugin
+* (bug 1284) Inline styles for diffs in Recent Changes RSS/Atom feeds
 
 
 === Caveats ===
diff --git a/includes/SpecialRecentchanges.php b/includes/SpecialRecentchanges.php
index f48a253cb2..8accbfb661 100644
--- a/includes/SpecialRecentchanges.php
+++ b/includes/SpecialRecentchanges.php
@@ -593,6 +593,7 @@ function rcFormatDiff( $row ) {
 			} else {
 				// Diff output fine, clean up any illegal UTF-8
 				$diffText = UtfNormal::cleanUp( $diffText );
+				$diffText = rcApplyDiffStyle( $diffText );
 			}
 			wfProfileOut( "$fname-dodiff" );
 		} else {
@@ -612,4 +613,32 @@ function rcFormatDiff( $row ) {
 	return $completeText;
 }
 
+/**
+ * Hacky application of diff styles for the feeds.
+ * Might be 'cleaner' to use DOM or XSLT or something,
+ * but *gack* it's a pain in the ass.
+ *
+ * @param string $text
+ * @return string
+ * @access private
+ */
+function rcApplyDiffStyle( $text ) {
+	$styles = array(
+		'diff'             => 'background-color: white;',
+		'diff-otitle'      => 'background-color: white;',
+		'diff-ntitle'      => 'background-color: white;',
+		'diff-addedline'   => 'background: #cfc; font-size: smaller;',
+		'diff-deletedline' => 'background: #ffa; font-size: smaller;',
+		'diff-context'     => 'background: #eee; font-size: smaller;',
+		'diffchange'       => 'color: red; font-wieght: bold;',
+	);
+	
+	foreach( $styles as $class => $style ) {
+		$text = preg_replace( "/(<[^>]+)class=(['\"])$class\\2([^>]*>)/",
+			"\\1style=\"$style\"\\3", $text );
+	}
+	
+	return $text;
+}
+
 ?>