(bug 19942) remove deprecated HTML attributes (cellpadding, align, valign) from Table...

diff --git a/includes/Pager.php b/includes/Pager.php
index 1d8b67a..9028f0e 100644 (file)
--- a/includes/Pager.php
+++ b/includes/Pager.php
@@ -815,7 +815,7 @@ abstract class TablePager extends IndexPager {
                $tableClass = htmlspecialchars( $this->getTableClass() );
                $sortClass = htmlspecialchars( $this->getSortHeaderClass() );
 
-               $s = "<table border='1' class=\"$tableClass\"><thead><tr>\n";
+               $s = "<table style='border:1;' class=\"$tableClass\"><thead><tr>\n";
                $fields = $this->getFieldNames();
 
                # Make table header
@@ -982,10 +982,10 @@ abstract class TablePager extends IndexPager {
                $links = $this->getPagingLinks( $linkTexts, $disabledTexts );
 
                $navClass = htmlspecialchars( $this->getNavClass() );
-               $s = "<table class=\"$navClass\" align=\"center\" cellpadding=\"3\"><tr>\n";
-               $cellAttrs = 'valign="top" align="center" width="' . 100 / count( $links ) . '%"';
+               $s = "<table class=\"$navClass\"><tr>\n";
+               $width = 100 / count( $links ) . '%';
                foreach ( $labels as $type => $label ) {
-                       $s .= "<td $cellAttrs>{$links[$type]}</td>\n";
+                       $s .= "<td style='width:$width;'>{$links[$type]}</td>\n";
                }
                $s .= "</tr></table>\n";
                return $s;

diff --git a/skins/common/shared.css b/skins/common/shared.css
index 9a8a1b5..07e5134 100644 (file)
--- a/skins/common/shared.css
+++ b/skins/common/shared.css
@@ -379,6 +379,14 @@ table.collapsed tr.collapsable {
 .TablePager {
        min-width: 80%;
        border-collapse: collapse;
+       margin: 0 auto;
+}
+.TablePager_nav {
+       margin: 0 auto;
+}
+.TablePager_nav td {
+       padding: 3px;
+       text-align: center;
 }
 .TablePager_nav a {
        text-decoration: none;