From 229b5aa755f304f94eea259d26a49cbdc7556c09 Mon Sep 17 00:00:00 2001
From: Happy-melon <happy-melon@users.mediawiki.org>
Date: Wed, 30 Mar 2011 17:37:25 +0000
Subject: [PATCH] (bug 19942) remove deprecated HTML attributes (cellpadding,
 align, valign) from TablePager.

---
 includes/Pager.php      | 8 ++++----
 skins/common/shared.css | 8 ++++++++
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/includes/Pager.php b/includes/Pager.php
index 1d8b67ae3d..9028f0e06b 100644
--- 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 9a8a1b5534..07e513435d 100644
--- 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;
-- 
2.20.1