MaxSem has uploaded a new change for review.
https://gerrit.wikimedia.org/r/67472
Change subject: Completely remove the $wgMFCustomLogos['logo'] stuff
......................................................................
Completely remove the $wgMFCustomLogos['logo'] stuff
It's been broken for a long time and doesn't comply with our current
designs anyway. Remove mw.png and don't default the mobile logo to it:
it's too small for our current usage of mobile logo and it's kinda pointless
to use a MW logo there.
Change-Id: I5501842553334e93cb67f213df7bc898fd7e2802
---
M MobileFrontend.php
D images/mw.png
M includes/MobileFrontend.body.php
M includes/skins/SkinMobile.php
4 files changed, 3 insertions(+), 22 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend
refs/changes/72/67472/1
diff --git a/MobileFrontend.php b/MobileFrontend.php
index 18db9c4..f358da3 100644
--- a/MobileFrontend.php
+++ b/MobileFrontend.php
@@ -165,9 +165,8 @@
$wgMFPhotoUploadWiki = null;
/**
- * Path to the logo used in the mobile view
- *
- * Should be 22px tall at most
+ * Path to the logo used in the login/signup form
+ * The standard height is 72px
*/
$wgMobileFrontendLogo = false;
diff --git a/images/mw.png b/images/mw.png
deleted file mode 100644
index a88b40a..0000000
--- a/images/mw.png
+++ /dev/null
Binary files differ
diff --git a/includes/MobileFrontend.body.php b/includes/MobileFrontend.body.php
index 4d97c54..7eee712 100644
--- a/includes/MobileFrontend.body.php
+++ b/includes/MobileFrontend.body.php
@@ -29,8 +29,6 @@
protected function beforePageDisplay( $out ) {
wfProfileIn( __METHOD__ );
- $this->setDefaultLogo();
-
$this->disableCaching();
$this->sendHeaders();
@@ -161,21 +159,5 @@
wfProfileOut( __METHOD__ );
return $contentHtml;
- }
-
- /**
- * Sets up the default logo image used in mobile view if none is set
- */
- public function setDefaultLogo() {
- global $wgMobileFrontendLogo, $wgExtensionAssetsPath,
$wgMFCustomLogos;
- wfProfileIn( __METHOD__ );
- if ( $wgMobileFrontendLogo === false ) {
- $wgMobileFrontendLogo = $wgExtensionAssetsPath .
'/MobileFrontend/images/mw.png';
- }
-
- if ( MobileContext::singleton()->isBetaGroupMember() && isset(
$wgMFCustomLogos['logo'] ) ) {
- $wgMobileFrontendLogo = $wgMFCustomLogos['logo'];
- }
- wfProfileOut( __METHOD__ );
}
}
diff --git a/includes/skins/SkinMobile.php b/includes/skins/SkinMobile.php
index e92131f..0d5cc07 100644
--- a/includes/skins/SkinMobile.php
+++ b/includes/skins/SkinMobile.php
@@ -181,7 +181,7 @@
$footerSitename = $this->msg( 'mobile-frontend-footer-sitename'
)->text();
- if ( is_array( $wgMFCustomLogos ) && isset(
$wgMFCustomLogos['copyright'] ) ) {
+ if ( isset( $wgMFCustomLogos['copyright'] ) ) {
$suffix = $wgMFTrademarkSitename ? ' ®' : '';
$sitename = Html::element( 'img', array(
'src' => $wgMFCustomLogos['copyright'],
--
To view, visit https://gerrit.wikimedia.org/r/67472
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5501842553334e93cb67f213df7bc898fd7e2802
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: MaxSem <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
