Toggle Function

[HighCaliber]

I am trying to create a toggle function for my forum.Here is my javasript code...

<script type="JavaScript">function toggle(obj1, obj2) {	var el = document.getElementById(obj1);	var el2 = document.getElementById(obj2);	if ( el.style.display != 'none' ) {		el.style.display = 'none';		el2.src = "Themes/Default/Images/category_expand.png";	}	else {		el.style.display = '';		el2.src = "Themes/Default/Images/category_shrink.png";	}}</script>

and here is what i am trying to toggle...

<a href="java script:toggle(1, si1)"><img alt="" border="0" src="Themes/Default/Images/category_shrink.png" id="si1" /></a><table cellspacing="1" class="Section_Middle" id="1"><tr><td>content to toggle</td></tr></table>

For some reason this will not work, when i go to validate it it says el2 has no properties. Can anyone figure how to fix this or show me a valid (W3C Standards) way to toggle two objects.

[aspnetguy]

your problem is here

<a href="java script:toggle(1, si1)">

change it to this

<a href="java script:toggle('1', 'si1')">

hmm it keeps putting a space between java and script, just take that out in your code.

[HighCaliber]

Thank you, it finally works i have been asking around on multiple forums trying to get this fixed and it was so simple. Now I feel stupid lol