Jdlrobson has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/67913


Change subject: Cleanup mf-common.less
......................................................................

Cleanup mf-common.less

Remove undocumented css (should be put in Mobile.css if a problem)
Move main page specific css to mainpage.css

Change-Id: Ic296baebaed1aa457374a44aa24335c2f40821dd
---
M less/common/mf-common.less
M less/mainpage/mainpage.less
M stylesheets/common/mf-common.css
M stylesheets/mainpage/mainpage.css
4 files changed, 44 insertions(+), 86 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend 
refs/changes/13/67913/1

diff --git a/less/common/mf-common.less b/less/common/mf-common.less
index f562935..85eff90 100644
--- a/less/common/mf-common.less
+++ b/less/common/mf-common.less
@@ -115,17 +115,7 @@
        }
 }
 
-.mwm-message.mwm-notice {
-       background: #FFFFFF;
-       font-size: 1.1em;
-       color: #000000;
-}
-.mwm-message.mwm-notice a {
-       text-decoration: underline;
-       font-weight: bold;
-       color: #000000;
-}
-
+// FIXME: Review .mwm-message
 .mwm-notice {
        padding: 5px;
        background: #dddddd;
@@ -133,6 +123,18 @@
        margin-top: 5px;
        text-align: center;
        border: 1px solid gray;
+
+       &.mwm-notice {
+               background: #FFFFFF;
+               font-size: 1.1em;
+               color: #000000;
+
+                a {
+                       text-decoration: underline;
+                       font-weight: bold;
+                       color: #000000;
+               }
+       }
 }
 
 #content {
@@ -184,9 +186,7 @@
 
 p,
 li,
-dl,
-#mainpage,
-#featured_article {
+dl {
        line-height: 1.65;
 }
 
@@ -217,15 +217,6 @@
        color: red;
 }
 
-#contentSub {
-       margin-bottom: 10px;
-}
-
-#featured_article .noprint,
-.today .noprint {
-       display: none;
-}
-
 img.thumbborder {
        border: 1px solid #cccccc;
 }
@@ -238,38 +229,14 @@
        float: left;
 }
 
-#content_wrapper.home h1 {
-       display: none;
-}
-
 #mw-mf-last-modified {
        display: block;
        font-size: .9em;
        margin-top: 1.5em;
 }
 
-#disableButtons {
-       text-align: center;
-}
-
-#disableButtons button {
-       background: url(images/buttonbg.gif) no-repeat center;
-       border: 0;
-       height: 23px;
-       width: 83px;
-}
-
-#disableButtons #backButton {
-       margin-top: 5em;
-       margin-bottom: 2em;
-}
-
 .mwm-message.mwm-notice {
        font-size: 1.1em;
-}
-
-#image_wrapper {
-       margin: 8px 0 8px -3px;
 }
 
 @media all and (max-width: 500px) {
diff --git a/less/mainpage/mainpage.less b/less/mainpage/mainpage.less
index 706a36b..cd6f81b 100644
--- a/less/mainpage/mainpage.less
+++ b/less/mainpage/mainpage.less
@@ -1,3 +1,14 @@
+#mainpage,
+#featured_article {
+       line-height: 1.65;
+}
+
+#featured_article, .today {
+       .noprint {
+               display: none;
+       }
+}
+
 body.page-Main_Page {
 
        table {
diff --git a/stylesheets/common/mf-common.css b/stylesheets/common/mf-common.css
index 9ccbb0a..fe2c560 100644
--- a/stylesheets/common/mf-common.css
+++ b/stylesheets/common/mf-common.css
@@ -96,16 +96,6 @@
   padding: 2px 4px;
   word-wrap: break-word;
 }
-.mwm-message.mwm-notice {
-  background: #FFFFFF;
-  font-size: 1.1em;
-  color: #000000;
-}
-.mwm-message.mwm-notice a {
-  text-decoration: underline;
-  font-weight: bold;
-  color: #000000;
-}
 .mwm-notice {
   padding: 5px;
   background: #dddddd;
@@ -113,6 +103,16 @@
   margin-top: 5px;
   text-align: center;
   border: 1px solid gray;
+}
+.mwm-notice.mwm-notice {
+  background: #FFFFFF;
+  font-size: 1.1em;
+  color: #000000;
+}
+.mwm-notice.mwm-notice a {
+  text-decoration: underline;
+  font-weight: bold;
+  color: #000000;
 }
 #content {
   position: relative;
@@ -158,9 +158,7 @@
 }
 p,
 li,
-dl,
-#mainpage,
-#featured_article {
+dl {
   line-height: 1.65;
 }
 pre {
@@ -184,13 +182,6 @@
 a.new:hover {
   color: red;
 }
-#contentSub {
-  margin-bottom: 10px;
-}
-#featured_article .noprint,
-.today .noprint {
-  display: none;
-}
 img.thumbborder {
   border: 1px solid #cccccc;
 }
@@ -202,32 +193,13 @@
 table.floatright {
   float: left;
 }
-#content_wrapper.home h1 {
-  display: none;
-}
 #mw-mf-last-modified {
   display: block;
   font-size: .9em;
   margin-top: 1.5em;
 }
-#disableButtons {
-  text-align: center;
-}
-#disableButtons button {
-  background: url(images/buttonbg.gif) no-repeat center;
-  border: 0;
-  height: 23px;
-  width: 83px;
-}
-#disableButtons #backButton {
-  margin-top: 5em;
-  margin-bottom: 2em;
-}
 .mwm-message.mwm-notice {
   font-size: 1.1em;
-}
-#image_wrapper {
-  margin: 8px 0 8px -3px;
 }
 @media all and (max-width: 500px) {
   .mobile div.tleft,
diff --git a/stylesheets/mainpage/mainpage.css 
b/stylesheets/mainpage/mainpage.css
index 3e8ee84..50838d4 100644
--- a/stylesheets/mainpage/mainpage.css
+++ b/stylesheets/mainpage/mainpage.css
@@ -1,3 +1,11 @@
+#mainpage,
+#featured_article {
+  line-height: 1.65;
+}
+#featured_article .noprint,
+.today .noprint {
+  display: none;
+}
 body.page-Main_Page table tr,
 body.page-Main_Page table td {
   display: block !important;

-- 
To view, visit https://gerrit.wikimedia.org/r/67913
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic296baebaed1aa457374a44aa24335c2f40821dd
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to