jenkins-bot has submitted this change and it was merged.
Change subject: Moving some content hiding from server-side to CSS
......................................................................
Moving some content hiding from server-side to CSS
This is a two part transition. First we will introduce the new CSS,
let people's old CSS caches expire, then we will remove the
corresponding HTML stripping code from MobileFrontend or the config
Change-Id: Ica45dfa10e6b510af13240ccf658172418b8ba26
---
M less/common/common.less
M less/common/hacks.less
M stylesheets/common/common.css
M stylesheets/common/hacks.css
4 files changed, 31 insertions(+), 12 deletions(-)
Approvals:
MaxSem: Looks good to me, approved
jenkins-bot: Verified
diff --git a/less/common/common.less b/less/common/common.less
index 49badad..76c2a9a 100644
--- a/less/common/common.less
+++ b/less/common/common.less
@@ -150,15 +150,20 @@
#content {
position: relative; // allow absolute positioned children e.g. edit link
-}
-#content table {
- /* following 5 rules are needed for scrolling */
- overflow: auto; /* for browsers that don't support overflow-y */
- overflow-y: hidden;
- overflow-x: auto;
- word-break: normal;
- display: block;
+ table {
+ /* following 5 rules are needed for scrolling */
+ overflow: auto; /* for browsers that don't support overflow-y */
+ overflow-y: hidden;
+ overflow-x: auto;
+ word-break: normal;
+ display: block;
+ }
+
+ // Hide the image magnification icon normally displayed in image
captions
+ div.magnify {
+ display: none;
+ }
}
table {
@@ -328,3 +333,7 @@
position: absolute;
}
}
+
+.nomobile {
+ display: none;
+}
diff --git a/less/common/hacks.less b/less/common/hacks.less
index 4062965..e1e54f7 100644
--- a/less/common/hacks.less
+++ b/less/common/hacks.less
@@ -54,9 +54,10 @@
clear: both !important;
}
-/* Fix for bug 53437 - nav boxes can disrupt reading experience on smaller
screens */
+/* Hide some on-wiki defined classes */
#content {
- .navbox, .vertical-navbox {
+ // Hide article badges, clean-up notices, stub notices, and navigation
boxes
+ .navbox, .vertical-navbox, .topicon, .metadata, div.magnify {
display: none;
}
}
diff --git a/stylesheets/common/common.css b/stylesheets/common/common.css
index 436045c..b3859ef 100644
--- a/stylesheets/common/common.css
+++ b/stylesheets/common/common.css
@@ -146,6 +146,9 @@
word-break: normal;
display: block;
}
+#content div.magnify {
+ display: none;
+}
table {
-moz-box-sizing: border-box;
-webkit-box-sizing: border-box;
@@ -276,3 +279,6 @@
.no-position-fixed .position-fixed {
position: absolute;
}
+.nomobile {
+ display: none;
+}
diff --git a/stylesheets/common/hacks.css b/stylesheets/common/hacks.css
index e082883..e9b7e2e 100644
--- a/stylesheets/common/hacks.css
+++ b/stylesheets/common/hacks.css
@@ -44,9 +44,12 @@
width: auto !important;
clear: both !important;
}
-/* Fix for bug 53437 - nav boxes can disrupt reading experience on smaller
screens */
+/* Hide some on-wiki defined classes */
#content .navbox,
-#content .vertical-navbox {
+#content .vertical-navbox,
+#content .topicon,
+#content .metadata,
+#content div.magnify {
display: none;
}
/* Revert to showing nav boxes on tablet/desktop */
--
To view, visit https://gerrit.wikimedia.org/r/86763
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ica45dfa10e6b510af13240ccf658172418b8ba26
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Kaldari <[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