From 9c3495dc1ea8966a5dafe83f29b88462e795b4bb Mon Sep 17 00:00:00 2001 From: Derk-Jan Hartman Date: Tue, 26 Jan 2016 20:35:29 +0100 Subject: [PATCH] Add mw-content-ltr|rtl to file redirect pages 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 | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/includes/skins/SkinTemplate.php b/includes/skins/SkinTemplate.php index bf260aa464..c3e3cea48f 100644 --- a/includes/skins/SkinTemplate.php +++ b/includes/skins/SkinTemplate.php @@ -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(); -- 2.20.1