From 6fb8848184b63a27957f171807507c715fec99e2 Mon Sep 17 00:00:00 2001
From: Tim Starling <tstarling@users.mediawiki.org>
Date: Thu, 17 Aug 2006 11:05:13 +0000
Subject: [PATCH] TablePager: fixes for IE (doesn't like coloured ridge
 borders) and Opera (doesn't like td+td+td+td+td). Added row highlighting on
 hover.

---
 includes/Pager.php      |  5 ++++-
 skins/common/common.css | 13 +++++++++----
 skins/monobook/main.css | 13 +++++++++----
 3 files changed, 22 insertions(+), 9 deletions(-)

diff --git a/includes/Pager.php b/includes/Pager.php
index de361b3cdc..4a276ad4ba 100644
--- a/includes/Pager.php
+++ b/includes/Pager.php
@@ -447,6 +447,8 @@ abstract class TablePager extends IndexPager {
 		
 		$s = "<table border='1' class=\"$tableClass\"><thead><tr>\n";
 		$fields = $this->getFieldNames();
+
+		# Make table header
 		foreach ( $fields as $field => $name ) {
 			if ( strval( $name ) == '' ) {
 				$s .= "<th>&nbsp;</th>\n";
@@ -504,7 +506,8 @@ abstract class TablePager extends IndexPager {
 			if ( $formatted == '' ) {
 				$formatted = '&nbsp;';
 			}
-			$s .= "<td>$formatted</td>\n";
+			$class = 'TablePager_col_' . htmlspecialchars( $field );
+			$s .= "<td class=\"$class\">$formatted</td>\n";
 		}
 		$s .= "</tr>\n";
 		return $s;
diff --git a/skins/common/common.css b/skins/common/common.css
index a34624b7b6..e630b77e8c 100644
--- a/skins/common/common.css
+++ b/skins/common/common.css
@@ -417,11 +417,16 @@ table.multipageimage td {
 */
 .TablePager_nav a { text-decoration: none; }
 .TablePager { border-collapse: collapse; }
-.TablePager, .TablePager td, .TablePager th { border: 2px outset #666666; }
-.TablePager td, .TablePager th { background-color: #eeeeff }
+.TablePager, .TablePager td, .TablePager th { 
+	border: 0.15em solid #777777;
+	padding: 0 0.15em 0 0.15em;
+}
+.TablePager th { background-color: #eeeeff }
+.TablePager td { background-color: #ffffff }
+.TablePager tr:hover td { background-color: #eeeeff }
 
 .imagelist td, .imagelist th { white-space: nowrap }
-.imagelist td + td { background-color: #ffffff }
-.imagelist td + td + td + td + td { white-space: normal }
+.imagelist .TablePager_col_links { background-color: #eeeeff }
+.imagelist .TablePager_col_img_description { white-space: normal }
 .imagelist th.TablePager_sort { background-color: #ccccff }
 
diff --git a/skins/monobook/main.css b/skins/monobook/main.css
index 8b52ee0d9b..95b4c7353a 100644
--- a/skins/monobook/main.css
+++ b/skins/monobook/main.css
@@ -1476,10 +1476,15 @@ table.multipageimage td {
 */
 .TablePager_nav a { text-decoration: none; }
 .TablePager { border-collapse: collapse; }
-.TablePager, .TablePager td, .TablePager th { border: 2px outset #666666; }
-.TablePager td, .TablePager th { background-color: #eeeeff }
+.TablePager, .TablePager td, .TablePager th { 
+	border: 0.15em solid #777777;
+	padding: 0 0.15em 0 0.15em;
+}
+.TablePager th { background-color: #eeeeff }
+.TablePager td { background-color: #ffffff }
+.TablePager tr:hover td { background-color: #eeeeff }
 
 .imagelist td, .imagelist th { white-space: nowrap }
-.imagelist td + td { background-color: #ffffff }
-.imagelist td + td + td + td + td { white-space: normal }
+.imagelist .TablePager_col_links { background-color: #eeeeff }
+.imagelist .TablePager_col_img_description { white-space: normal }
 .imagelist th.TablePager_sort { background-color: #ccccff }
-- 
2.20.1