jenkins-bot has submitted this change and it was merged.
Change subject: Blindly remove simple, opera mini and opera mobile stylesheets
......................................................................
Blindly remove simple, opera mini and opera mobile stylesheets
Who knows what these undocumented rules are here for
Nothing looks vital in these
Change-Id: I9971836cbcea2f9a4c7b2857dcb881fa75386ed5
---
M includes/DeviceDetection.php
D stylesheets/devices/operamini.css
D stylesheets/devices/operamobile.css
D stylesheets/devices/simple.css
M tests/DeviceDetectionTest.php
5 files changed, 0 insertions(+), 70 deletions(-)
Approvals:
MaxSem: Looks good to me, approved
JGonera: Looks good to me, but someone else must approve
jenkins-bot: Verified
diff --git a/includes/DeviceDetection.php b/includes/DeviceDetection.php
index f1adc84..704151f 100644
--- a/includes/DeviceDetection.php
+++ b/includes/DeviceDetection.php
@@ -227,7 +227,6 @@
),
'netfront' => array (
'view_format' => 'html',
- 'css_file_name' => 'simple',
),
'nokia' => array (
'view_format' => 'html',
@@ -235,11 +234,9 @@
),
'operamini' => array (
'view_format' => 'html',
- 'css_file_name' => 'operamini',
),
'operamobile' => array (
'view_format' => 'html',
- 'css_file_name' => 'operamobile',
),
'palm_pre' => array (
'view_format' => 'html',
diff --git a/stylesheets/devices/operamini.css
b/stylesheets/devices/operamini.css
deleted file mode 100644
index a031b6a..0000000
--- a/stylesheets/devices/operamini.css
+++ /dev/null
@@ -1,27 +0,0 @@
-.header {
- padding-top: 9px !important;
-}
-
-table { /* table[style] not supported */
- float: none !important;
- margin-left: 0 !important;
- width: 100% !important;
- overflow: auto;
-}
-
-.mwm-notice {
- text-align: left;
-}
-
-.mwm-message.mwm-notice {
- display: none;
-}
-
-a.arrow {
- float: right;
- margin-right: 5px;
-}
-
-span.idx {
- display: none;
-}
diff --git a/stylesheets/devices/operamobile.css
b/stylesheets/devices/operamobile.css
deleted file mode 100644
index 0fe711c..0000000
--- a/stylesheets/devices/operamobile.css
+++ /dev/null
@@ -1,7 +0,0 @@
-table {
- overflow-y: scroll;
-}
-
-tbody {
- height: 200px;
-}
diff --git a/stylesheets/devices/simple.css b/stylesheets/devices/simple.css
deleted file mode 100644
index 138180d..0000000
--- a/stylesheets/devices/simple.css
+++ /dev/null
@@ -1,30 +0,0 @@
-.header,
-#footer,
-#content_wrapper {
- margin: 0 2px;
-}
-
-/* TODO: duplicate see psp.css */
-.nav {
- border: 0;
- border-bottom: 2px solid #6d84b4;
- border-top: 2px solid #6d84b4;
- height: auto;
- clear: both;
- padding: 5px 0 5px 0;
- margin: 0;
-}
-
-.mwm-notice {
- text-align: left;
-}
-
-.mwm-message.mwm-notice {
- display: none;
-}
-
-a.arrow {
- float: right;
- margin-right: 5px;
-}
-
diff --git a/tests/DeviceDetectionTest.php b/tests/DeviceDetectionTest.php
index 1869657..6b5fa1b 100644
--- a/tests/DeviceDetectionTest.php
+++ b/tests/DeviceDetectionTest.php
@@ -89,9 +89,6 @@
array( 'iphone', 'mobile.device.iphone' ),
array( 'kindle', 'mobile.device.kindle' ),
array( 'blackberry', 'mobile.device.blackberry' ),
- array( 'netfront', 'mobile.device.simple' ),
- array( 'operamini', 'mobile.device.operamini' ),
- array( 'operamobile', 'mobile.device.operamobile' ),
array( 'nokia', 'mobile.device.nokia' ),
array( 'wml', '' )
);
--
To view, visit https://gerrit.wikimedia.org/r/72146
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9971836cbcea2f9a4c7b2857dcb881fa75386ed5
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>
Gerrit-Reviewer: JGonera <[email protected]>
Gerrit-Reviewer: MaxSem <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
