Eugene233 has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/378867 )

Change subject: Use CSS to set the width of cells at Special:Newsletters
......................................................................

Use CSS to set the width of cells at Special:Newsletters

Currently the width of cells on the newletter tablepage is adjusted
using code which is deprecated. This has been done alternatively by using CSS.

Bug: T175645
Change-Id: I0b45ce9ef92c0101cc2de68777d55c222354a8eb
---
M includes/specials/pagers/NewsletterTablePager.php
M modules/ext.newsletter.newsletters.styles.css
2 files changed, 26 insertions(+), 16 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Newsletter 
refs/changes/67/378867/1

diff --git a/includes/specials/pagers/NewsletterTablePager.php 
b/includes/specials/pagers/NewsletterTablePager.php
index 59f35f6..d623a6e 100644
--- a/includes/specials/pagers/NewsletterTablePager.php
+++ b/includes/specials/pagers/NewsletterTablePager.php
@@ -339,22 +339,6 @@
         */
        public function getCellAttrs( $field, $value ) {
                $ret = parent::getCellAttrs( $field, $value );
-               // @todo use CSS, not inline HTML
-               switch ( $field ) {
-                       case 'nl_name':
-                               $ret['width'] = '20%';
-                               break;
-                       case 'nl_desc':
-                               $ret['width'] = '40%';
-                               break;
-                       case 'subscriber_count':
-                               $ret['width'] = '5%';
-                               break;
-                       case 'action':
-                               $ret['width'] = '20%';
-                               break;
-               }
-
                return $ret;
        }
 
diff --git a/modules/ext.newsletter.newsletters.styles.css 
b/modules/ext.newsletter.newsletters.styles.css
index c3cd71f..09ffbb4 100644
--- a/modules/ext.newsletter.newsletters.styles.css
+++ b/modules/ext.newsletter.newsletters.styles.css
@@ -52,6 +52,32 @@
        cursor: default;
 }
 
+.mw-special-Newsletters
+.mw-datatable
+.TablePager_col_nl_name {
+    width: 20%;
+}
+
+}
+
+.mw-special-Newsletters
+.mw-datatable
+.TablePager_col_nl_desc {
+    width: 40%;
+}
+
+
+.mw-special-Newsletters
+.mw-datatable
+.TablePager_col_subscriber_count {
+    width: 5%;
+}
+
+.mw-special-Newsletters
+.mw-datatable .TablePager_col_action {
+    width: 20%;
+}
+
 .mw-special-Newsletters .mw-datatable,
 .mw-special-Newsletters .mw-datatable td,
 .mw-special-Newsletters .mw-datatable th {

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0b45ce9ef92c0101cc2de68777d55c222354a8eb
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Newsletter
Gerrit-Branch: master
Gerrit-Owner: Eugene233 <agboreug...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to