На картинках стоит onClick, вызывающий функцию, которая слоям меняет свойства display. Почему то срабатывает со 2 раза стабильно. Причём после 2 раза, уже нормально. На простых линках такое ставил - работает нормально - сразу. Может кто сталкивался и подскажет в чём проблема... 

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

Сейчас исправлюсь.
Вызов идёт так:  




Теперь ява-скрипт:




hideAll() - ставит всем слоям, кроме указанного, display: none; а указанному соотвественно display: block;

Добавлено через 9 минут и 10 секунд
Сейчас исправлюсь.
Вызов идёт так:  




Теперь ява-скрипт:




hideAll() - ставит всем слоям, кроме указанного, display: none; а указанному соотвественно display: block;

Сегодня нет времени моделировать ситуацию - разве что ты вышлешь полный вариант на ICQ-адрес. Но с ходу могу сказать, что в функции show ничего кроме одной стороки просто ненужно, и if, который берет значение стиля отображения, в особенности. Попробуй

function show(obj) 
{
  document.getElementById(obj).style.display = 'block';
}
Вторую функцию тоже надо бы разгрузить от избыточности - в этом тоже может быть проблема. Что-то типа:
function hideAll(obj) {
 for (var i = 0; i < divs.length; i++) {
   document.getElementById(divs[i]).style.display = 'none';
 }
 show(obj);
}

Это сообщение отредактировал(а) Rigel - 27.8.2007, 16:33

Дело в том, что если текущий слой открыт и мы кликаем на его открытие, он должен закрыться, поэтому в show() и стоят условия

В том, что я написал, он и закроется - смотри внимательно. Они ВСЕ закроются. А потом откроется один.

Так все слои изначально открыты? Если они все закрыты, то зачем "пробегать" по ним в цикле и закрывать?

Всё слои изначально закрыты. Бегу по всем, потому что неизвестно, какой из них открыт в данный момент(и открыт ли вообще)

Добавлено через 1 минуту и 47 секунд

Если слой открыт и мы на нём щелкаем, то он должен закрыться. А в вашем коде, сначала все закроются, потом 1 ОБЯЗАТЕЛЬНО откроется.

Надо попробовать запоминать в переменной  текущий открытый слой, чтоб не бегать по всем, может прокатит

Раз уж точно известно, что все слои изначально закрыты, то можно сделать проще. Можно сохранять каждый раз при открытии слоя ссылку на него в какой-нибудь переменной. Таким образом всегда будет известно, какой слой стал активным (открытым/закрытым) в момент последнего клика.

Сделал так:



Теперь не бегу циклом по всем. Результат тот же что и раньше :(

Добавлено через 2 минуты и 32 секунды
Опа, даже больше скажу! Стало хуже работать - теперь всегда со 2 раза :( Думаю, надо как то по другому show() делать. Думаю, в нём косяк

Micher, я имел ввиду что-нибудь, вроде этого:

Вместо hideAll используйте show...

Опа, даже больше скажу!  теперь всегда со 2 раза    

А смотрите случайно не в Опере? А то у Оперы свои приколы с невидимыми элементами. Во всяком случае в кэш она невидимые картинки не складывает.

Смотрю и в Опере, и IE и в Firefox, везде работает одинакого