Mooeypoo has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/361712 )

Change subject: Special:RecentChanges - Move legend into results
......................................................................

Special:RecentChanges - Move legend into results

** This affects all RecentChanges pages, not just RCFilters **

Move the legend from the fieldset area to the results area.

Bug: T168043
Change-Id: I40454b194fe3975d8d4ac13d96774bfa5b5d5a46
---
M includes/specials/SpecialRecentchanges.php
M 
resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.ChangesListWrapperWidget.js
M resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FormWrapperWidget.js
3 files changed, 18 insertions(+), 18 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/12/361712/1

diff --git a/includes/specials/SpecialRecentchanges.php 
b/includes/specials/SpecialRecentchanges.php
index 5ec2064..662eaa3 100644
--- a/includes/specials/SpecialRecentchanges.php
+++ b/includes/specials/SpecialRecentchanges.php
@@ -521,6 +521,7 @@
 
                $userShowHiddenCats = $this->getUser()->getBoolOption( 
'showhiddencats' );
                $rclistOutput = $list->beginRecentChangesList();
+               $rclistOutput .= $this->makeLegend();
                foreach ( $rows as $obj ) {
                        if ( $limit == 0 ) {
                                break;
@@ -588,7 +589,7 @@
                $nondefaults = $opts->getChangedValues();
 
                $panel = [];
-               $panel[] = $this->makeLegend();
+               // $panel[] = $this->makeLegend();
                $panel[] = $this->optionsPanel( $defaults, $nondefaults, 
$numRows );
                $panel[] = '<hr />';
 
diff --git 
a/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.ChangesListWrapperWidget.js
 
b/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.ChangesListWrapperWidget.js
index c7754be..bb29f08 100644
--- 
a/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.ChangesListWrapperWidget.js
+++ 
b/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.ChangesListWrapperWidget.js
@@ -92,7 +92,8 @@
                var conflictItem,
                        $message = $( '<div>' )
                                .addClass( 
'mw-rcfilters-ui-changesListWrapperWidget-results' ),
-                       isEmpty = $changesListContent === 'NO_RESULTS';
+                       isEmpty = $changesListContent === 'NO_RESULTS',
+                       collapseCookieName = 'changeslist-state';
 
                this.$element.toggleClass( 'mw-changeslist', !isEmpty );
                if ( isEmpty ) {
@@ -130,6 +131,19 @@
                        // Apply highlight
                        this.applyHighlight();
 
+                       // Collapse legend
+                       // see 
resources/src/mediawiki.special/mediawiki.special.changelist.legend.js
+                       this.$element.find( '.mw-changeslist-legend' )
+                               .makeCollapsible( {
+                                       collapsed: mw.cookie.get( 
collapseCookieName ) === 'collapsed'
+                               } )
+                               .on( 'beforeExpand.mw-collapsible', function () 
{
+                                       mw.cookie.set( collapseCookieName, 
'expanded' );
+                               } )
+                               .on( 'beforeCollapse.mw-collapsible', function 
() {
+                                       mw.cookie.set( collapseCookieName, 
'collapsed' );
+                               } );
+
                        // Make sure enhanced RC re-initializes correctly
                        mw.hook( 'wikipage.content' ).fire( this.$element );
                }
diff --git 
a/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FormWrapperWidget.js 
b/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FormWrapperWidget.js
index 4e1ae97..2258312 100644
--- a/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FormWrapperWidget.js
+++ b/resources/src/mediawiki.rcfilters/ui/mw.rcfilters.ui.FormWrapperWidget.js
@@ -117,8 +117,7 @@
         * Clean up the old-style show/hide that we have implemented in the 
filter list
         */
        mw.rcfilters.ui.FormWrapperWidget.prototype.cleanUpFieldset = function 
() {
-               var $namespaceSelect = this.$element.find( '#namespace' ),
-                       collapseCookieName = 'changeslist-state';
+               var $namespaceSelect = this.$element.find( '#namespace' );
 
                this.$element.find( 
'.rcshowhideoption[data-feature-in-structured-ui=1]' ).each( function () {
                        // HACK: Remove the text node after the span.
@@ -138,19 +137,5 @@
                        $namespaceSelect.closest( 'tr' ).detach();
                        this.$element.find( '.mw-tagfilter-label' ).closest( 
'tr' ).detach();
                }
-
-               // Collapse legend
-               // see 
resources/src/mediawiki.special/mediawiki.special.changelist.legend.js
-               this.$element.find( '.mw-changeslist-legend' )
-                       .makeCollapsible( {
-                               collapsed: mw.cookie.get( collapseCookieName ) 
=== 'collapsed'
-                       } )
-                       .on( 'beforeExpand.mw-collapsible', function () {
-                               mw.cookie.set( collapseCookieName, 'expanded' );
-                       } )
-                       .on( 'beforeCollapse.mw-collapsible', function () {
-                               mw.cookie.set( collapseCookieName, 'collapsed' 
);
-                       } );
-
        };
 }( mediaWiki ) );

-- 
To view, visit https://gerrit.wikimedia.org/r/361712
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I40454b194fe3975d8d4ac13d96774bfa5b5d5a46
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Mooeypoo <mor...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to