Вот так правильно? function facechange (objName) { if ( $(objName).css('display') == 'none' ) { $(objName).animate({height: 'show'}, 400); $('span').text(' - '); } else { $(objName).animate({height: 'hide'}, 200); $('span').text(' + '); } } Я, к сожалению, не слишком хорошо разбираюсь в javascript. И еще есть вопрос не будет ли этот код слишком загружать страницу? Дело в том, что я хочу использовать его для страницы FAQ (Часто задаваемые вопросы), то есть, таких дивов на странице будет около 25? И вообще, чем этот код отличается или лучше этого: function show_squad (id) { if (document.getElementById("span_" + id).style.display == 'none') { document.getElementById("span_" + id).style.display = "block"; } else { document.getElementById("span_" + id).style.display = "none"; } } Большое спасибо!