jenkins-bot has submitted this change and it was merged. Change subject: i18n: Make noarticletext direction aware ......................................................................
i18n: Make noarticletext direction aware Currently https://fa.wikipedia.org/wiki/MediaWiki:TEST.js?uselang=fa is displaying "noarticletext" in LTR which is wrong. Bug: T60685 Change-Id: I4dc76dc1c5134c0dbe0988d6d0a3286a0e52242f --- M includes/page/Article.php 1 file changed, 7 insertions(+), 1 deletion(-) Approvals: Amire80: Looks good to me, approved Ladsgroup: Looks good to me, but someone else must approve jenkins-bot: Verified diff --git a/includes/page/Article.php b/includes/page/Article.php index e58ffc8..54db19c 100644 --- a/includes/page/Article.php +++ b/includes/page/Article.php @@ -1270,7 +1270,13 @@ $text = wfMessage( 'noarticletext-nopermission' )->plain(); } - $outputPage->addWikiText( "<div class='noarticletext'>\n$text\n</div>" ); + $dir = $this->getContext()->getLanguage()->getDir(); + $lang = $this->getContext()->getLanguage()->getCode(); + $outputPage->addWikiText( Xml::openElement( 'div', array( + 'class' => "noarticletext mw-content-$dir", + 'dir' => $dir, + 'lang' => $lang, + ) ) . "\n$text\n</div>" ); } } -- To view, visit https://gerrit.wikimedia.org/r/228750 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4dc76dc1c5134c0dbe0988d6d0a3286a0e52242f Gerrit-PatchSet: 4 Gerrit-Project: mediawiki/core Gerrit-Branch: master Gerrit-Owner: Gerrit Patch Uploader <[email protected]> Gerrit-Reviewer: Amire80 <[email protected]> Gerrit-Reviewer: Ebrahim <[email protected]> Gerrit-Reviewer: Gerrit Patch Uploader <[email protected]> Gerrit-Reviewer: Ladsgroup <[email protected]> Gerrit-Reviewer: Nikerabbit <[email protected]> Gerrit-Reviewer: Siebrand <[email protected]> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
