IE, NS & Firerox Problem - DIVs, document.layers and getElementById

[InvisibleMan]

Thanks in Advance for any help on this - its truely sending my
head in loops... so I appreciate your efforts!

okay, I have a javascript listed below that drops down submenus
contained within:
<div class="small" style="display: none" id="menu1_menu" >

when the heading is clicked:
<a href=.. onClick="expand Div('menu1')">

- This works perfect in IE, NS6 (I Think), but not NS7 or
FireFox. I know my issues lay in the 'var Browsers = ' and
document.layers , getElementById. The menu is opened and set
using the function setMenu() called elsewhere... menu1 is the
default, but it does store a cookie if this changes to menu2,
menu3 etc...

Essentially i'd like this to work for all browsers - in fact its
essential it does :( - Please stop me from loosing anymore hair
- Regards Lee
var ua = navigator.userA gent;
var opera = /opera [56789]|opera\/[56789]/i.test(ua);
var ie = !opera && /msie [56789]/i.test(ua);
// preventing opera to be identified as ie
var moz = !opera && /mozilla\/[56789]/i.test(ua);
// preventing opera to be identified as mz

function getCookie(NameO fCookie)
{

if (document.cooki e.length > 0)
{

begin = document.cookie .indexOf(NameOf Cookie+"=");
if (begin != -1)
{

begin += NameOfCookie.le ngth+1;
end = document.cookie .indexOf(";", begin);
if (end == -1) end = document.cookie .length;
return unescape(docume nt.cookie.subst ring(begin, end)); }
}
return "menu1";

}

function setCookie(NameO fCookie, value, expiredays)
{

var ExpireDate = new Date ();
ExpireDate.setT ime(ExpireDate. getTime() + (expiredays * 24 *
3600 * 1000));

document.cookie = NameOfCookie + "=" + escape(value) +
((expiredays == null) ? "" : "; expires=" +
ExpireDate.toGM TString());
}

function closeall()
{
var objs;

if (ie)
{
objs = document.all.ta gs("DIV");
}
else if (ua)
{
objs = document.getEle mentById("DIV") ;
}
else if (document.layer s)
{
objs = document.layers["DIV"];
}
else
{
alert('Non supported browser');
}

for (var i=0; i<objs.length; i++) {
if (objs[i].className == "small")
{

objs[i].style.display = "none";
}
}
}

function expandDiv(tahw) {
setCookie("menu ", tahw, 7);
closeall();

what = tahw + "_menu"

if (document.getEl ementById(what) .style.display == "none") {
document.getEle mentById(what). style.display = "";
} else {
document.getEle mentById(what). style.display = "none";
}
}

function setmenu()
{
var menuvar;
menuvar = getCookie("menu ");
expandDiv(menuv ar);
}

----------------------------------------------
Posted with NewsLeecher v2.0 Beta 5
* Binary Usenet Leeching Made Easy
* http://www.newsleecher.com/?usenet
----------------------------------------------

[Fred Oz]

InvisibleMan wrote:
[...]

- This works perfect in IE, NS6 (I Think), but not NS7 or
FireFox. I know my issues lay in the 'var Browsers = ' and
document.layers , getElementById. The menu is opened and set
Then that's what I'll concentrate on fixing.
using the function setMenu() called elsewhere... menu1 is the
default, but it does store a cookie if this changes to menu2,
menu3 etc...

Essentially i'd like this to work for all browsers - in fact its
essential it does :( - Please stop me from loosing anymore hair
- Regards Lee
var ua = navigator.userA gent;
var opera = /opera [56789]|opera\/[56789]/i.test(ua);
var ie = !opera && /msie [56789]/i.test(ua);
// preventing opera to be identified as ie
var moz = !opera && /mozilla\/[56789]/i.test(ua);
// preventing opera to be identified as mz [...]

Get rid of the crappy browser detection. Try:

if (document.getEl ementById) {
objs = document.getEle mentById('DIV') ;
} else if (document.all) {
objs = document.all['DIV'];
} else if (document.layer s) {
objs = document.all['DIV'];
}

...

[...]
for (var i=0; i<objs.length; i++) {
It is more efficient to get the length just once:

for (var i=0, var len=<objs.lengt h; i<len; i++) {
...

[...] if (document.getEl ementById(what) .style.display == "none") {
document.getEle mentById(what). style.display = "";
} else {
document.getEle mentById(what). style.display = "none";
}
}

And you'd better add feature detection (as above) here too, else the
non-getElementById browsers will fail.

That may not fix all your issues, but it should get you going again.

--
Fred

[InvisibleMan]

Thanks for that okay, i've done the following but keep getting
errors... i'm getting what your trying to say... but its not
working and causing errors... Help

function closeall(){
var objs;

if (document.getEl ementById) {
objs = document.getEle mentById('DIV') ;
} else if (document.all) {
objs = document.all['DIV'];
} else if (document.layer s) {
objs = document.all['DIV'];
} else {
alert('Non supported browser');
}
for (var i=0; i<objs.length; i++) {
if (objs[i].className == "small")
{
objs[i].style.display = "none";
}
}
}

AND

function expandDiv(tahw) {
setCookie("thin gsmenu", tahw, 7);
closeall();

what = tahw + "_menu"

if (document.getEl ementById(what) .style.display ==
"none") {
document.getEle mentById(what). style.display = "")
} else if (document.all) {
document.all['DIV'](what).style.di splay = "none");
} else if (document.layer s) {
document.all['DIV'](what).style.di splay = "none");
}
else {
document.getEle mentById(what). style.display = "none";
}
}

----------------------------------------------
Posted with NewsLeecher v2.0 Beta 5
* Binary Usenet Leeching Made Easy
* http://www.newsleecher.com/?usenet
----------------------------------------------

[Fred Oz]

InvisibleMan wrote:
[...]

if (document.getEl ementById) {
objs = document.getEle mentById('DIV') ;

Sorry, I think what you are looking for is
document.getEle mentsByTagName:

if (document.getEl ementsByTagName ) {
objs = document.getEle mentsByTagName( 'DIV');

and that will return a collection.

--
Fred