Commit b7ddc3a4 authored by Dries's avatar Dries
Browse files

- Bugfix: removed 'width="100%"' attributes from table tags. Patch by Al.

  "IE has a bug with tables inside divs with width=100%. I see no reason why
  the forum tables should be 100% - they are pretty much naturally padded to
  100% as soon as anyone puts any content in them. Removing them doesn't
  affect functionality and improves layout compatibility on IE."
parent cb2d27c7
...@@ -529,7 +529,7 @@ function forum_forum_list($forums, $parents, $tid) { ...@@ -529,7 +529,7 @@ function forum_forum_list($forums, $parents, $tid) {
} }
$t[] = l(t("Discussion forum"), "forum"); $t[] = l(t("Discussion forum"), "forum");
$output .= "<table border=\"0\" cellpadding=\"5\" width=\"100%\">\n"; $output .= "<table border=\"0\" cellpadding=\"5\">\n";
$output .= " <tr><th colspan=\"3\" align=\"left\">".implode(" : ", array_reverse($t)) ."</th>"; $output .= " <tr><th colspan=\"3\" align=\"left\">".implode(" : ", array_reverse($t)) ."</th>";
if ($forums) { if ($forums) {
$output .= "<th>". t("topics") ."</th><th>". t("posts") ."</th><th>". t("last post") ."</th></tr>"; $output .= "<th>". t("topics") ."</th><th>". t("posts") ."</th><th>". t("last post") ."</th></tr>";
...@@ -600,7 +600,7 @@ function forum_topic_list($tid, $topics, $sortby, $forum_per_page, $offset) { ...@@ -600,7 +600,7 @@ function forum_topic_list($tid, $topics, $sortby, $forum_per_page, $offset) {
global $id, $status, $user, $pager_total; global $id, $status, $user, $pager_total;
if ($topics) { if ($topics) {
$output .= "<table border=\"0\" cellpadding=\"5\" cellspacing=\"5\" width=\"100%\">\n"; $output .= "<table border=\"0\" cellpadding=\"5\" cellspacing=\"5\">\n";
$output .= " <tr><th>&nbsp;</th><th>&nbsp;</th><th>". t("topic") ."</th><th>". t("replies") ."</th><th>". t("posted") ."</th><th>". t("last reply") ."</th></tr>"; $output .= " <tr><th>&nbsp;</th><th>&nbsp;</th><th>". t("topic") ."</th><th>". t("replies") ."</th><th>". t("posted") ."</th><th>". t("last reply") ."</th></tr>";
foreach ($topics as $topic) { foreach ($topics as $topic) {
......
...@@ -529,7 +529,7 @@ function forum_forum_list($forums, $parents, $tid) { ...@@ -529,7 +529,7 @@ function forum_forum_list($forums, $parents, $tid) {
} }
$t[] = l(t("Discussion forum"), "forum"); $t[] = l(t("Discussion forum"), "forum");
$output .= "<table border=\"0\" cellpadding=\"5\" width=\"100%\">\n"; $output .= "<table border=\"0\" cellpadding=\"5\">\n";
$output .= " <tr><th colspan=\"3\" align=\"left\">".implode(" : ", array_reverse($t)) ."</th>"; $output .= " <tr><th colspan=\"3\" align=\"left\">".implode(" : ", array_reverse($t)) ."</th>";
if ($forums) { if ($forums) {
$output .= "<th>". t("topics") ."</th><th>". t("posts") ."</th><th>". t("last post") ."</th></tr>"; $output .= "<th>". t("topics") ."</th><th>". t("posts") ."</th><th>". t("last post") ."</th></tr>";
...@@ -600,7 +600,7 @@ function forum_topic_list($tid, $topics, $sortby, $forum_per_page, $offset) { ...@@ -600,7 +600,7 @@ function forum_topic_list($tid, $topics, $sortby, $forum_per_page, $offset) {
global $id, $status, $user, $pager_total; global $id, $status, $user, $pager_total;
if ($topics) { if ($topics) {
$output .= "<table border=\"0\" cellpadding=\"5\" cellspacing=\"5\" width=\"100%\">\n"; $output .= "<table border=\"0\" cellpadding=\"5\" cellspacing=\"5\">\n";
$output .= " <tr><th>&nbsp;</th><th>&nbsp;</th><th>". t("topic") ."</th><th>". t("replies") ."</th><th>". t("posted") ."</th><th>". t("last reply") ."</th></tr>"; $output .= " <tr><th>&nbsp;</th><th>&nbsp;</th><th>". t("topic") ."</th><th>". t("replies") ."</th><th>". t("posted") ."</th><th>". t("last reply") ."</th></tr>";
foreach ($topics as $topic) { foreach ($topics as $topic) {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment