Сообщение от poorking
|
Зачем там везде eval ¿
add+="<div id='pic"+i+"' style='z-index:"+(500-Math.round(180*Math.cos((Math.PI*((photos[i].arct % 360)+90))/180)+180))+";position:absolute;left:"+(photos[i].width/2+x)+"px;top:"+(photos[i].height/2+y)+"px;width:"+photos[i].width+"px;height:"+photos[i].height+"px;'><img class='gElement' alt='"+photos[i].alt+"' width="+photos[i].width+" height="+photos[i].height+" lang='"+name+"' src='"+photos[i].path+"' style='position:absolute;top:0px;left:0px;'></div>";}
dec.innerHTML+=add;
Зачем такие вот маневры тоже не очень ясно, еще видел как вы устанавливаете таймеры не получив ссылку на таймер чтоб потом его удалить, значит не удаляете, а там они на каждом событии, То есть если я мышкой повожу по галерее, то у меня уже таймеров 100 будет работать и так далее. Ну это чисто теоретически.
|
а таймер сам вообще-то очищается, после того как выполняется, и никакие ссылки не нужны, посмотрите что у этих таймеров, таймер 50 мс, и плюс там проверка условия для запуска ещё одного таймера, то есть лишних не будет, а маневры с add+= нужно чтобы постоянно не тыкать в документ элементы через innerHTML, сначала все элементы подготовить, а потом добавить!
eval, от тут да, возможно промах, просто боялся запутаться
с видимостью переменных, и перестраховался, главное, что работает, и таких строчек там немного)