>
document.link1.className = 'sel';
The way you are using to access the link element is incorrect. The document object doesn't have a property named 'link1', it has a link element having an id of 'link1'. The correct way of doing things would be:
var e = document.getElementById('link1');
e.className = 'sel';
A slightly better(definitely not the best!) way of doing it would be:
<script type="text/javascript">
function activate(link)
{
var e = document.getElementById(link);
var e3 = document.getElementById(link3);
var e4 = document.getElementById(link4);
if(!e || !e3 || !e4)
return;
if(link == 'link1')
{
e.className = 'sel';
e.className = 'nav';
}
else if(link == 'link2')
{
e.className = 'sel';
e.className = 'nav';
}
e3.className = e4.className = 'nav';
}
</script> I don't accept change. I don't deserve to live.
Happiness corrupts people.
Failing to value the lives of others cheapens your own.
