Merge "RCFilters: Also set toplinks-collapsed class if no cookie"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Tue, 26 Sep 2017 13:39:12 +0000 (13:39 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Tue, 26 Sep 2017 13:39:13 +0000 (13:39 +0000)
includes/specials/SpecialRecentchanges.php

index 522a0a6..6ce1968 100644 (file)
@@ -620,7 +620,8 @@ class SpecialRecentChanges extends ChangesListSpecialPage {
                        if ( $this->isStructuredFilterUiEnabled() ) {
                                // Check whether the widget is already collapsed or expanded
                                $collapsedState = $this->getRequest()->getCookie( 'rcfilters-toplinks-collapsed-state' );
-                               $collapsedClass = $collapsedState === 'collapsed' ? 'mw-rcfilters-toplinks-collapsed' : '';
+                               // Note that an empty/unset cookie means collapsed, so check for !== 'expanded'
+                               $collapsedClass = $collapsedState !== 'expanded' ? 'mw-rcfilters-toplinks-collapsed' : '';
 
                                $contentTitle = Html::rawElement( 'div',
                                        [ 'class' => 'mw-recentchanges-toplinks-title ' . $collapsedClass ],