изменение класса по onclick

[ureech]

Здравствуйте.Есть ссылки по которым открывается iframe.Если нет клика вывожу картинку
Ссылка:

<div id="my"><a   target="player"  href="{$video.video}" > {$video.title|truncate:30} </a> </div>

фрейм и картинка:

<div id="invis" ><iframe name="player"  src=""  scrolling="no" ></iframe> </div>
     <div id="vis"><img  src="vidio/image/02.png" /> </div>

Написал ф-цию ( первую в жизни ), что бы по клику пряталась картинка и открывался фрейм.

$(document).ready(function(){
	$('#vis img').addClass('test');
	$('#invis').addClass('invis');
	$('#my').click(function(){
	$('#vis img').removeClass('test').addClass('invis');
	$('#invis').addClass('vis');
	});
});

но работает, только если сначало кликнуть по первой ссылке. Подскажите как поправить.

[BETEPAH]

Не совсем понятно как эти два предложения друг другу противоречат:

но работает, только если сначало кликнуть по первой ссылке

что бы по клику пряталась картинка

[ureech]

Открывается страница. Фрейма нет. Есть картинка.Есть N ссылок.Кликаю по первой. Картинка ушла, фрейма пришла. Но если сначало кликаю по любой другой ссылке, то не работает.

[BETEPAH]

в приведенном коде видна только одна ссылка. Кроме того, в js Вы вешаете слушатель клика тоже только на одну ссылку (даже не на ссылку, а на один div#my)

[ureech]

Это динамика.Полный вариант

{foreach key=id item=video from=$videos}
<div id="my"><a   target="player"  href="{$video.video}" > {$video.title|truncate:30} </a> </div>
{/foreach}

И нет разницы в данном случае где слушатель. Пробовал и так и так.

[ureech]

Всё правильно, с id получается только одна ссылка. Поменял на class. Всё заработало. Спасибо за помощь.

[BETEPAH]

Это динамика.Полный вариант

если бы привели сразу полный вариант, можно было бы без наводящих вопросов сразу сказать, что id должен быть уникальным на странице, поэтому скрипт вешает только на первый найденный id