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 a9c3a67..470b150 100644 (file)
--- 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 1727da0..1c3a0fb 100644 (file)
--- 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>';
        }
 }