помогите оптимизировать под Mozilu i NN

Код

<script language="JavaScript"> var img1 = new Image(); img1.src = "plus.gif"; var img2 = new Image(); img2.src = "minus.gif"; function doOutline()      {   var srcId, srcElement, targetElement;   srcElement = window.event.srcElement;   if (srcElement.className.toUpperCase() == "LEVEL1" || srcElement.className.toUpperCase() == "FAQ") {          srcID = srcElement.id.substr(0, srcElement.id.length-1);          targetElement = document.all(srcID + "s");          srcElement = document.all(srcID + "i");     if (targetElement.style.display == "none") {                              targetElement.style.display = "";                  if (srcElement.className == "LEVEL1") srcElement.src = img2.src;        } else {                  targetElement.style.display = "none";                  if (srcElement.className == "LEVEL1") srcElement.src = img1.src;      }   } } </script>

Вызов функции:

Код

<BODY OnMouseUp="doOutline()" BGCOLOR=#D4D0C8 LEFTMARGIN=0 TOPMARGIN=0 MARGINWIDTH=0 MARGINHEIGHT=0 >

М	Жми на кнопку code в следующий раз

Это сообщение отредактировал(а) Sardar - 27.10.2005, 11:04

Свойства document.all не существует в мозилле. Присваивай каждому элементу ID и выбирай по нему.

Явно скрипт от какого то дерева. Мозилла (Опера и остальные) любят скрипты написанные по стандартам W3C, жаль что "матёрые" програмисты этого не читают.

Во первых window.event не существует, обьект передаёться первым параметром функции. Раз функция взываеться из inline обработчика, значит ты должен сам передать обьект-событие: OnMouseUp="doOutline(event)"

srcElement не существует, есть target, хотя по мне srcElement более наглядное имя, но W3C с этим не согласна.

document.all заменяем на document.getElementById(), это справедливо и для ИЕ.

В остальном всё будет работать.
Добавлено @ 11:11

Цитата(sergej @ 27.10.2005, 10:06)

Свойства document.all не существует в мозилле.

Кстати существует, просто Мозилла будет кидать warning'и, но работать будет

Заменил document.all на document.getElementById(), но по прежнему не работает под Moziloi i NN.

Код

<script language="JavaScript"> .............................          srcID = srcElement.id.substr(0, srcElement.id.length-1);          targetElement = document.getElementById(srcID + "s");          srcElement = document.getElementById(srcID + "i"); ......................... </script> <BODY OnMouseUp="doOutline(event)" BGCOLOR=#D4D0C8 LEFTMARGIN=0 TOPMARGIN=0 MARGINWIDTH=0 MARGINHEIGHT=0>

Это сообщение отредактировал(а) Aliance - 4.11.2005, 19:20

!	Пользуемся кнопкой код!

Cfcoder сразу скажу, это было глупо вырезать верхнюю и нижнюю часть кода, показав только getElementById. Да, как минимум две последние строчки должны работать правильно везде, если ты доволен этим ответом

Приведи весь изменённый код. Повторюсь, работу с событиями нужно переделать, т.к. уверен что srcElement ты попрежнему от window.event достаёшь.

Вообще читай и выполняй рекомендации полностью, а не то что кажеться самым простым

Я не очень силен в JavaScripte! Пожалуйста помогите переделать!

Код

<script language="JavaScript"> var img1 = new Image(); img1.src = "plus.gif"; var img2 = new Image(); img2.src = "minus.gif"; function doOutline()      {   var srcId, srcElement, targetElement;   srcElement = window.event.srcElement;   if (srcElement.className.toUpperCase() == "LEVEL1" || srcElement.className.toUpperCase() == "FAQ") {          srcID = srcElement.id.substr(0, srcElement.id.length-1);          targetElement = document.getElementById(srcID + "s");          srcElement = document.getElementById(srcID + "i");     if (targetElement.style.display == "none") {                              targetElement.style.display = "";                  if (srcElement.className == "LEVEL1") srcElement.src = img2.src;        } else {                  targetElement.style.display = "none";                  if (srcElement.className == "LEVEL1") srcElement.src = img1.src;      }   } } </script>

Уже разобрался!!!