Replace 'TablePager' CSS class exclusively by `mw-datatable`
authorVolker E <volker.e@wikimedia.org>
Sat, 19 Jan 2019 02:48:13 +0000 (18:48 -0800)
committerLegoktm <legoktm@member.fsf.org>
Wed, 13 Mar 2019 07:29:31 +0000 (07:29 +0000)
`mw-datatable` has already been in use and should be the main class
for styling TablePager tables.
`mw-datatable` also includes the equivalent padding of former
`.TablePager th` & `td`, therefore removing it.

Bug: T214218
Change-Id: I2025b2ede87035e15f0280a6fe5efddec174bcca

includes/pager/TablePager.php
resources/src/mediawiki.pager.tablePager/TablePager.less

index 71b1ad7..d94104b 100644 (file)
@@ -152,7 +152,7 @@ abstract class TablePager extends IndexPager {
 
                $tableClass = $this->getTableClass();
                $ret = Html::openElement( 'table', [
-                       'class' => "mw-datatable $tableClass" ]
+                       'class' => " $tableClass" ]
                );
                $ret .= Html::rawElement( 'thead', [], Html::rawElement( 'tr', [], "\n" . $s . "\n" ) );
                $ret .= Html::openElement( 'tbody' ) . "\n";
@@ -266,10 +266,11 @@ abstract class TablePager extends IndexPager {
        }
 
        /**
+        * TablePager relies on `mw-datatable` for styling, see T214208
         * @return string
         */
        protected function getTableClass() {
-               return 'TablePager';
+               return 'mw-datatable';
        }
 
        /**
index c7eae2c..1cccb88 100644 (file)
@@ -5,17 +5,10 @@
 
 @import 'mediawiki.mixins';
 
-.TablePager {
+// TablePager uses `.mw-datatable` and is loaded in the right order by RL
+.mw-datatable {
        min-width: 80%;
 
-       // Remove as soon as `.mw-datatable` receives this `padding`, see T214208
-       td,
-       th {
-               padding: 0.2em 0.4em;
-       }
-}
-
-.mw-datatable {
        &-is-sorted a {
                background-position: left center;
                background-repeat: no-repeat;