Click to See Complete Forum and Search --> : Table and drop down boxes broken in IE8. Any clues?


HairyArse
06-07-2009, 02:02 PM
Hey guys,

Just uploaded a new version of my website and various things aren't working in IE8, but the main problem I'm having is on this page:

http://www.allaboutthegames.co.uk/forum/

I've narrowed the problem down to the thread title field which includes a file (I'll paste) below that basically generates a drop down menu if a forum thread has more than one page.

This works fine in IE7, Firefox and Opera but breaks IE8. Does anyone have any ideas why?



<form>
<a class="forum" href="forum_thread.php?thread_id=<? echo("$thread_id");?>"><? echo("$thread_title"); ?></a>
<? //if there is more than 1 page then display the drop down menu with page numbers
if ($number_of_pages > 1)
{?>
<br />
<select name="URL" onchange="window.location.href=this.form.URL.options[this.form.URL.selectedIndex].value">
<option value ="Page" selected>Page</option>
<?
for($i=1; $i<=$number_of_pages; $i++)
{
$startposition = (($i-1)*30); ?>
<option value="../forum/forum_thread.php?thread_id=<? echo ("$thread_id"); ?>&start=<? echo ("$startposition"); ?>"><? echo("$i"); ?></option>
<?
}?>
</select>
&nbsp;<a class="forum" href="forum_thread.php?thread_id=<? echo("$thread_id&start=$last_page");?>">Last page</a>
<?
}
?></form>

johanafm
06-07-2009, 07:50 PM
At a quick glance the code you posted seems fine apart from incorrect use of ampersands: &. Whenever you wish to use an actual ampersand in HTML, you put &amp; in the code.

Apart from that, I'd start by running the whole code through a validator.w3.org and fix the 950 errors it finds! And while you're at it, might as well change margin-botton to margin-bottom in the CSS.

Once those errors are gone, it could just be you get the expected results out of IE8.