0

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

function displayBlock(hallId)
{
 document.getElementsByClassName("hidden_b").style.display = "none";
 document.getElementById("hidden_block_"+hallId).style.display = "block";
}

<div id="hidden_block_'.$hall_row['hall_id'].'" class="hidden_b" style="display:none;"></div>
<a href="javascript:void(0);" onclick="displayBlock('.$hall_row['hall_id'].');">Клик</a>

<div id="hidden_block_'.$hall_row['hall_id'].'" class="hidden_b" style="display:none;"></div>
<a href="javascript:void(0);" onclick="displayBlock('.$hall_row['hall_id'].');">Клик</a>

<div id="hidden_block_'.$hall_row['hall_id'].'" class="hidden_b" style="display:none;"></div>
<a href="javascript:void(0);" onclick="displayBlock('.$hall_row['hall_id'].');">Клик</a>
2
  • В функции onclick вы передаете непонятно что, а не hallId. К тому же у блоков вместо id непонятно что написано. 30 сен 2013 в 9:56
  • это выдержка из php скрипта, там передается простое число из массива $hall_row
    – ANTiK
    30 сен 2013 в 10:02

1 ответ 1

3

document.getElementsByClassName возвращает массив элементов. У массива нет свойства style. Чтобы изменить стиль у всех элементов массива нужно перебирать их в цикле и менять у каждого в отдельности. Или, что гораздо проще, использовать jQuery, где делается всё то же самое, но прозрачно для программиста. В jQuery эта строка выглядела бы так:

$('.hidden_b').css('display', 'none')

Или ещё проще:

$('.hidden_b').hide()

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.