jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/405824 )
Change subject: Always load user styles on mobile user pages
......................................................................
Always load user styles on mobile user pages
Previously the styles were only loaded if the user existed
and they had no user page. While we want to do this for the
content we don't want to do this for styles.
Bug: T182162
Change-Id: I6e474562ba5f2c704364c9d7b49f982f7bab8a18
---
M includes/MobileFrontend.body.php
1 file changed, 5 insertions(+), 4 deletions(-)
Approvals:
Pmiazga: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/MobileFrontend.body.php b/includes/MobileFrontend.body.php
index 29a4549..82de6e2 100644
--- a/includes/MobileFrontend.body.php
+++ b/includes/MobileFrontend.body.php
@@ -66,14 +66,15 @@
// on whether the user is the owner of the page or not.
if ( $title->inNamespace( NS_USER ) && !$title->isSubpage() ) {
$pageUserId = User::idFromName( $title->getText() );
+
+ $out->addModuleStyles( [
+ 'mediawiki.ui.icon',
+ 'mobile.userpage.styles',
'mobile.userpage.icons'
+ ] );
if ( $pageUserId && !$title->exists() ) {
$pageUser = User::newFromId( $pageUserId );
$contentHtml = self::getUserPageContent(
$out, $pageUser );
- $out->addModuleStyles( [
- 'mediawiki.ui.icon',
- 'mobile.userpage.styles',
'mobile.userpage.icons'
- ] );
}
}
--
To view, visit https://gerrit.wikimedia.org/r/405824
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6e474562ba5f2c704364c9d7b49f982f7bab8a18
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <jrob...@wikimedia.org>
Gerrit-Reviewer: Pmiazga <pmia...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
