Hello there

I'm having trouble with my navigation system and hoping someone can help me: it's a show all/hide all kind of drop down menu and I keep getting this error when I click on one of the menu items:

Error: 'style.display' is null or not an object - line 37

this is line 37

        if (id5.style.display == ""){

and this is the rest of the script:

<script language="JavaScript" type="text/JavaScript">
// Quick browser check !
ie4 = (document.all) ? true : false;

function showSubCats(targetId, id2, id3, id4, id5, id6){
    if (ie4){

        target = document.all(targetId);
        id2 = document.all(id2);
        id3 = document.all(id3);
        id4 = document.all(id4);
        id5 = document.all(id5);
        id6 = document.all(id6);

        if (target.style.display == "none"){
            target.style.display = "";
        } else {
            target.style.display = "none";
        }

        if (id2.style.display == ""){
            id2.style.display = "none";
        }

        if (id3.style.display == ""){
            id3.style.display = "none";
        }

        if (id4.style.display == ""){
            id4.style.display = "none";
        }

        if (id5.style.display == ""){
            id5.style.display = "none";
        }

        if (id6.style.display == ""){
            id6.style.display = "none";
        }

    }//End If
    else {
    document.location = "?NavID=" + targetId
    }
}

</script>

many thanks

Adam

it's possible to view it in action at http://80.82.139.249/st-sar/navigation3.asp

thanks

Adam

It seems to work now, doesn't it?

Why do you make this a IE4 only page (or in fact, an IE only page as document.all works or all IE browsers)?? Hiding and displaying stuff like that can easily be done cross browser, so other standards compliant browsers can benefit from it as well.

Cheers,

Imar
---------------------------------------
Imar Spaanjaars
Everyone is unique, except for me.
While typing this post, I was listening to: Slow Love by Prince (Track 7 from the album: Sign O The Times (Disk 1))

Hi Adam,

You have two rows with id="PR" and two with id="CU" in the page.

All ids in a page must be unique.

HTH,

Chris

fantastic guys - thanks a million