Add mw-content-ltr|rtl to file redirect pages
authorDerk-Jan Hartman <hartman.wiki@gmail.com>
Tue, 26 Jan 2016 19:35:29 +0000 (20:35 +0100)
committerMatthias Mullie <git@mullie.eu>
Wed, 3 May 2017 13:24:27 +0000 (15:24 +0200)
28ea73bc3a07a4588f1b5fde4f5c41522eaa9518 added NS_FILE to the list,
because most of the content is user language. That does not
really apply for #REDIRECT, however.

As a side effect, this also makes the redirect arrows visible on File
redirect pages, because their CSS depends on these classes.

Another side effect is that now "additional" content will show for the
redirect as reported broken in T29857.

Bug: T60955
Change-Id: I6b14c17a4b74ff5964ea61554aa3afb197d206bf

includes/skins/SkinTemplate.php

index bf260aa..c3e3cea 100644 (file)
@@ -269,11 +269,13 @@ class SkinTemplate extends Skin {
                # An ID that includes the actual body text; without categories, contentSub, ...
                $realBodyAttribs = [ 'id' => 'mw-content-text' ];
 
-               # Add a mw-content-ltr/rtl class to be able to style based on text direction
-               # when the content is different from the UI language, i.e.:
-               # not for special pages or file pages AND only when viewing
-               if ( !in_array( $title->getNamespace(), [ NS_SPECIAL, NS_FILE ] ) &&
-                       Action::getActionName( $this ) === 'view' ) {
+               # Add a mw-content-ltr/rtl class to be able to style based on text
+               # direction when the content is different from the UI language (only
+               # when viewing)
+               # Most information on special pages and file pages is in user language,
+               # rather than content language, so those will not get this
+               if ( Action::getActionName( $this ) === 'view' &&
+                       ( !$title->inNamespaces( NS_SPECIAL, NS_FILE ) || $title->isRedirect() ) ) {
                        $pageLang = $title->getPageViewLanguage();
                        $realBodyAttribs['lang'] = $pageLang->getHtmlCode();
                        $realBodyAttribs['dir'] = $pageLang->getDir();