Раскрывающийся список не работает в Firefox

[allmoney.ws]

Раскрывающаяся информация по городам не работает в Mozilla Firefox. Что не так?

http://www.activeheat.ru/contacts.php

В IE вс? прекрасно работает

Помогите, пожалуйста, разобраться в ч?м дело.

[yopopt]

В этом проблема:

function display(elem) {
 if (elem.style.display != "block") {
 elem.style.display = "block";
 //event.srcElement.innerText = "Ñêðûòü";
 }
 else {
 elem.style.display = "none";
 }
 return false;
}

Вместо elem.style.display используйте document.getElementById(elem).style.display

[allmoney.ws]

Код изменил на

 function display(elem) {
 if (document.getElementById(elem).style.display != "block") { 
 document.getElementById(elem).style.display = "block";
 //event.srcElement.innerText = "Скрыть";
 }
 else {
 document.getElementById(elem).style.display = "none";
 }
 return false;
 }

По прежнему не работает в Firefox, и перестало работать в IE

[yopopt]

Кавычки забыли: onclick="return display('msk')"

[allmoney.ws]

Спасибо, заработало, но теперь в Firefox если сворачивать информацию по городу обратно - оста?тся много пустого места... С этим как бороться?

[yopopt]

Попробуйте вместо установки display:block просто сбрасывать значение display. Вот так ...display = "";

Но чтобы это заработало Вам прид?тся избавиться от класса "address"(или от display в н?м), прописав вместо него style="display:none;". Вообще, по хорошему, изначально надо оставлять вс? видимым, а яваскриптом скрывать. Иначе параноики ничего не увидят.

[allmoney.ws]

избавиться от класса "address"(или от display в н?м), прописав вместо него style="display:none;".

Не понял, как сделать

по хорошему, изначально надо оставлять вс? видимым, а яваскриптом скрывать. Иначе параноики ничего не увидят.

А не подскажите, может знаете пример данной реализации, чтобы я подсмотрел?

[yopopt]

 function display(elem) {
 if (document.getElementById(elem).style.display != "") { 
 document.getElementById(elem).style.display = "";
 //event.srcElement.innerText = "Скрыть";
 }
 else {
 document.getElementById(elem).style.display = "none";
 }
 return false;
 }

В коде надо убрать class="address".

А не подскажите, может знаете пример данной реализации, чтобы я подсмотрел?

На событи onload вешаете вызов этой функции передавая ей нужные id. Самое простое:

window.onload = function() {
 document.getElementById("msk").style.display = "none";
 document.getElementById("spb").style.display = "none";
 и тд...
}

[allmoney.ws]

Отлично, спасибо огромное, вс? работает и в IE, и в Firefox, но теперь при загрузке страницы видно как информация загружается несв?рнутой и сворачивается. Можно ли как-то избавиться от этого эффекта?

[Emm]

А так:

function SwitchMenu(obj){
 if(document.getElementById){
 var el = document.getElementById(obj);
 if(el.style.display != "block"){
 el.style.display = "block";
 }else{
 el.style.display = "none";
 }
 }
}

ссылка

update: к слою: id="msk" style="display:none"

[yopopt]

Отлично, спасибо огромное, вс? работает и в IE, и в Firefox, но теперь при загрузке страницы видно как информация загружается несв?рнутой и сворачивается. Можно ли как-то избавиться от этого эффекта?

Нет. Однако можно попробовать уменьшить время между е? появлением и выполнением скрипта. Вместо onload ждать появления элемента:

function hide() { 
 var msk = document.getElementById("msk");
 if (msk != null && msk !== undefined) {
 msk.style.display = "none";
 }
 var spb = document.getElementById("spb");
 if (spb != null && spb !== undefined) {
 spb.style.display = "none";
 }
// и тд...
 else { setTimeout(function() { hide(); }, 0); }
}
hide();

Должно работать.

[allmoney.ws]

При добавлении новых городов приходится добавлять новую строку типа

document.getElementById("city").style.display = "none";

в файл .js

А можно как-то это автоматизировать, чтобы не добавлять идентичные строки?

[yopopt]

Можно обойти все строки с id:

window.onload = function() {
	var trs = document.getElementById("cities").getElementsByTagName('tr');
	for (var i = 0; i < trs.length; i++) {
		(trs[i].getAttribute('id')) ? trs[i].style.display = "none" : 0;
	}
}

Для этого необходимо в родительскую таблицу добавить id(у меня id="cities").

Москва

[Колос]

Но чтобы это заработало Вам прид?тся избавиться от класса "address"(или от display в н?м), прописав вместо него style="display:none;".

Спасибо, большое. Про style.display="" уже нарыл в интернете. Но не помогало - тоже было

Если кто знает почему так работает, можете объяснить смысл?