From: Bartosz Dziewoński <matma.rex@gmail.com>
Date: Tue, 26 Nov 2013 22:13:51 +0000 (+0100)
Subject: Wrap changes lists in <div class="mw-changeslist" />
X-Git-Tag: 1.31.0-rc.0~17705^2
X-Git-Url: http://git.cyclocoop.org/%22%20.%20generer_url_ecrire%28%22brouteur%22%2C%28%24id_rubrique%20?a=commitdiff_plain;h=48cc8109c80f8e1fd1fc4af565c119911cdb198d;p=lhc%2Fweb%2Fwiklou.git

Wrap changes lists in <div class="mw-changeslist" />

This affects Watchlist, RecentChanges and RecentChangesLinked.

A <div class="mw-changeslist-empty" /> is already used on all of these
when the list isn't generated (somewhat inconsistently coded, though).

Bug: 8844
Change-Id: I1ab2a54e22f318713c2211d63b29c184751a3589
---

diff --git a/includes/changes/ChangesList.php b/includes/changes/ChangesList.php
index a9c3a677bc..470b150c6c 100644
--- a/includes/changes/ChangesList.php
+++ b/includes/changes/ChangesList.php
@@ -162,7 +162,7 @@ class ChangesList extends ContextSource {
 		$this->rclistOpen = false;
 		$this->getOutput()->addModuleStyles( 'mediawiki.special.changeslist' );
 
-		return '';
+		return '<div class="mw-changeslist">';
 	}
 
 	/**
@@ -246,11 +246,9 @@ class ChangesList extends ContextSource {
 	 * @return String
 	 */
 	public function endRecentChangesList() {
-		if ( $this->rclistOpen ) {
-			return "</ul>\n";
-		} else {
-			return '';
-		}
+		$out = $this->rclistOpen ? "</ul>\n" : '';
+		$out .= '</div>';
+		return $out;
 	}
 
 	/**
diff --git a/includes/changes/EnhancedChangesList.php b/includes/changes/EnhancedChangesList.php
index 1727da02aa..1c3a0fbd96 100644
--- a/includes/changes/EnhancedChangesList.php
+++ b/includes/changes/EnhancedChangesList.php
@@ -43,7 +43,7 @@ class EnhancedChangesList extends ChangesList {
 			'mediawiki.icon',
 		) );
 
-		return '';
+		return '<div class="mw-changeslist">';
 	}
 
 	/**
@@ -689,6 +689,6 @@ class EnhancedChangesList extends ChangesList {
 	 * @return string
 	 */
 	public function endRecentChangesList() {
-		return $this->recentChangesBlock() . parent::endRecentChangesList();
+		return $this->recentChangesBlock() . '</div>';
 	}
 }