Пара проблем:
- Вы забыли
}
. - Вы вызываете методы jQuery для элементов, которые не обернуты в объект jQuery.Вам нужно будет сделать:
$(itemName.getElementsByTagName("span")[0]).show("slow");
(обратите внимание на упаковку).Методы jQuery волшебным образом не расширяют элементы по умолчанию, объект должен быть перенесен первым.
Обратите внимание, что эта версия работает .
EDIT :
В качестве альтернативы, вы можете использовать второй параметр конструкции (области действия) jQuery и сократить этот код:
function showy(itemName) {
$('span:first',itemName).show("slow");
}
function closy(itemName) {
$('span:first',itemName).hide("slow");
}
EDITv2
Воспитанный Хуанхороший момент, вы должны также отделить JavaScript с разметкой.Под этим я подразумеваю избегать использования атрибутов on * элементов и хранить привязки во внешнем файле .js или тегах <script>
.например,
<head>
...
<script src="http://path.to/jquery.js">
<script>
$(function(){ // execute once the document is ready (onload="below_code()")
// bind to the buttons' hover events
// find them by the "button" and "white" class names
$('.button.white').hover(function(){ // hover event (onmouseover="below_code()")
// find the first span within the link that triggered the event
$('span:first',this).show('slow');
},function(){ // mouse out event (onmouseout="below_code()")
// likewise, find first span
$('span:first',this).hide('slow');
});
});
</script>
...
</head>
<body>
...
<a href="#" class="button white" id="button1">
<span id="spanToShow">SHOW: this text </span>
on hover
</a>
...
</body>