jenkins-bot has submitted this change and it was merged.

Change subject: Adding default styling for tables
......................................................................


Adding default styling for tables

Also removing some redundant styling for infoboxes and
consolidating the two table.infobox styles.

Bug: 49959
Change-Id: Ib3376651b8bbca3fb5b5ff8e3020efc435be0408
---
M less/common/common.less
M less/common/enwp.less
2 files changed, 15 insertions(+), 23 deletions(-)

Approvals:
  Jdlrobson: Looks good to me, approved
  Awjrichards: Looks good to me, but someone else must approve
  jenkins-bot: Verified



diff --git a/less/common/common.less b/less/common/common.less
index 51dde3d..f71f3b0 100644
--- a/less/common/common.less
+++ b/less/common/common.less
@@ -187,10 +187,8 @@
        }
 
        table {
-               caption {
-                       display: block;
-                       text-align: left;
-               }
+               margin: 1em 0;
+               border: 1px @grayLight solid;
 
                /* following 4 rules are needed for scrolling */
                overflow: auto; /* for browsers that don't support overflow-y */
@@ -200,7 +198,18 @@
 
                td,
                th {
-                padding: 3px;
+                       padding: 3px;
+                       border: 1px @grayLight solid;
+               }
+
+               th {
+                       background-color: #f2f2f2;
+                       text-align: center;
+               }
+
+               caption {
+                       display: block;
+                       text-align: left;
                }
        }
 
diff --git a/less/common/enwp.less b/less/common/enwp.less
index f046ea9..aace709 100644
--- a/less/common/enwp.less
+++ b/less/common/enwp.less
@@ -22,19 +22,6 @@
        margin-bottom: 10px;
 }
 
-table.infobox {
-       background: #f9f9f9;
-}
-
-table.infobox caption {
-       display: block;
-}
-
-table.infobox th,
-table.infobox td {
-       border: 0;
-}
-
 /* force portals to use a one column layout on mobile */
 .mobile .portal-column-left-wide,
 .mobile .portal-column-left,
@@ -49,12 +36,8 @@
 }
 
 table.infobox {
+       background: #f9f9f9;
        text-align: center !important;
-}
-
-table.infobox td,
-table.infobox th {
-       border: 1px solid #cccccc;
 }
 
 /* bug 34878: Set an optimal width for a column.

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib3376651b8bbca3fb5b5ff8e3020efc435be0408
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Kaldari <[email protected]>
Gerrit-Reviewer: Awjrichards <[email protected]>
Gerrit-Reviewer: JGonera <[email protected]>
Gerrit-Reviewer: Jdlrobson <[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

Reply via email to