Het is jammer dat je je menu in JavaScript doet (theoretisch is het mogelijk dat iemand dat niet ziet wanneer die JavaScript uit heeft gezet in zijn browser), en je menu kan je met CSS er mooi opmaken met <ul> en <li> (want dat is een menu eigenlijk, een lijst van items).
En de opmerking van Jelmer natuurlijk :)
Voor de rest is het goed gedaan.
Edit:
Je hebt ook nog wat foutjes waardoor je HTML niet valideert: Je DOCTYPE moet helemaal bovenaan staan, die komt nu na de <html><head>-tags. De andere zijn minder belangrijk, maar je weet de weg naar de HTML Validator zelf wel te vinden (en de CSS-validator natuurlijk ook wel) :)
@ Kasper : mijn menu (linken) komt uit de database en is geen JavaScript :)
Het is dit:
<?php
function showgegevens()
{
$sql = "Select * From navigatie Order By nav_id";
$data = mysql_query($sql) or die(mysql_error());
for($a=0;$a < mysql_num_rows($data);$a++)
{
$row = mysql_fetch_array($data);
// de linken onder elkaar
echo "<a href=".$row['5']." alt=".$row['3']." target=".$row['6']." onMouseOver=\"window.status='colours-of-snuggle-dogs'; return true;\" onMouseOut=\"window.status='colours-of-snuggle-dogs'; return true;\">".$row['1']."</a><br>";
}
}
?>
Ik heb het DOCTYPE verandert en zou nu OK moeten zijn :)
Geweldig deze hulp en tips :)
Dank jullie wel!!
Dit stukje is toch JS: onMouseOver=\"window.status='colours-of-snuggle-dogs'; return true;\" onMouseOut=\"window.status='colours-of-snuggle-dogs'; return true;\"?
Anyway, dat zou beter kunnen denk ik. Bijvoorbeeld door de waarden (die in jouw geval overigens niet erg beschrijvend zijn) te retourneren via een array en ze dan in een loopje te zetten.
En zoals je het nu hebt zou je er ook een unordered list (<ul>) van kunnen maken, maar het is de vraag of je dat wilt natuurlijk :)
<?php
function showgegevens()
{
$sql = "Select * From navigatie Order By nav_id";
$data = mysql_query($sql) or die(mysql_error());
echo '<ul>';
for($a=0;$a < mysql_num_rows($data);$a++)
{
$row = mysql_fetch_array($data);
// de linken onder elkaar
echo "<li><a href=".$row['5']." alt=".$row['3']." target=".$row['6']." onMouseOver=\"window.status='colours-of-snuggle-dogs'; return true;\" onMouseOut=\"window.status='colours-of-snuggle-dogs'; return true;\">".$row['1']."</a></li>";
}
echo '</ul>';
}
?>
En de ul en de li kan je vervolgens opmaken zoals je wilt (bijvoorbeeld de opmaak die je nu ook hebt).
De bolletjes haal je weg met list-style: none; en met margin en padding kan je de linkjes wat beter positioneren.