event.y+window.document.body.scrollTop не работает в firefox

[Юзер]

Подскажите, как сделать следующий (полагаю, очень старый) скрипт, чтобы он работал и в файр фоксе.

if ( ((event.y+window.document.body.scrollTop) < parseInt(document.all.mml.style.top)+2)  ||  ((event.x+window.document.body.scrollLeft) < parseInt(document.all.mml.style.left)+2 ) ||  ((event.x+window.document.body.scrollLeft) > parseInt(document.all.mml.style.left)+parseInt(document.all.mml.style.width)-2) )
	  {
		 for (var i=0; i<EL_QUANT; i++)
		 {document.all.submenu[i].style.visibility="hidden"; }
	   }

В ИЕ и Опере работает нормально. Правда при валидации говорит, что лучше использовать GetElementById вместо document.all. Но когда я делаю подобную замену - код перестает работать даже в ИЕ и Опере.

Это выпадающее меню таким образом реализовано.

[Vlad]

Замени document.all.mml.style.top на getElementById("mml").style.top

[Юзер]

Заменил. Пишет, что element has no properties.

document.getElementById("submenu") has no properties

Я пробовал и так писать:

document.getElementById("submenu["+i+"]")

- то же самое.

Полностью функция вот:

var EL_QUANT=5;
function openit(num)
{ for (var i=0; i<EL_QUANT; i++)
 {if (i!=num) {document.getElementById("submenu[i]").style.visibility="hidden";}
 else {document.getElementById("submenu[i]").style.visibility="visible";}}}

[yopopt]

Зачем вам квадратные скобки? У вас как выглядят id? Если они вида submenu1 и тд, то пишите так document.getElementById("submenu"+i).

[Юзер]

квадратные скобки написал потому, что они используются в старой версии скрипта:

document.all.submenu[i].style.visibility="hidden";

А в теле документа на самом деле просто

<div id="submenu" style="position:absolute; width:136px; left:146; height:103px; z-index:4; top:20px;" onMouseOut="closeit(1)">

Т.е. submenu идет без номера, но выполняется функция closeit(1) с определенным номером.

Если в странице подставляю номера для субменю в скобках - вываливает все подменю открытыми.

Короче - выше моего понимания пока что.

[yopopt]

То есть у Вас есть много блоков с одинаковым id="submenu"?

[Юзер]

Угу. 5 штук. Это выпадающее меню.

[yopopt]

Так нельзя. id должен быть уникальным для документа. Создайте внешний контейнер с id="submenu" и перебирайте вложенные дивы так:

var EL_QUANT=5;
function openit(num)
{ for (var i=0; i<EL_QUANT; i++)
 {if (i!=num) {document.getElementById("submenu").getElementsByTagName("div")[i].style.visibility="hidden";}
 else {document.getElementById("submenu").getElementsByTagName("div")[i].style.visibility="visible";}}}