for перебор массива

[Ratnikov310]

var el=document.getElementsByClassName('forum');
	for(var i=0; i<el.length; i++) {
	console.log(el[i]);
	el[i].addEventListener('click', gp('viewtopic.php?view='+el[i].getAttribute('to'), 'GET', ''));
	}

Не работает ничего

Суть кода: добавление обработчика ко всем элементам с классом forum.

Но ни в консоль, ни обработчика

Что я сделал не так

[hypnocolor]

Поместите скрипт в конец HTML-документа, перед закрывающим </body>.

[Ratnikov310]

у меня как-раз там

[hypnocolor]

Ну не знаю. Значит, ошиблись где-то ещё. Конкретно этот код — рабочий.

[Ratnikov310]

щас посметрел файл с этим скриптом:

Загрузка XHR - не работает

Локально - работает

В чем прикол тогда????

[hedgehog]

щас посметрел файл с этим скриптом:

Загрузка XHR - не работает

Локально - работает

В чем прикол тогда????

Когда отрабатывает цикл, элементы еще не загружены?

[Ratnikov310]

Так если скрипт грузится самым последним, то по идее все элементы загружены.

И еще, поправил вместо gp - alert, то что-то получается нето - сразу в цикле вылетают алерты, которые должны были вылетать ТОЛЬКО после щелчка по элементу к которыму добавлен обработчик...

[hedgehog]

Так если скрипт грузится самым последним, то по идее все элементы загружены.

XHR?

Контент подгружаете динамически, что-ли?

[Ratnikov310]

Контент подгружаете динамически, что-ли?

Ну да

[hedgehog]

Ну да

Судя по смайлам могу предположить, что сами уже все поняли

[Ratnikov310]

Ну пришлось уж в каждый <li> вставлять onclick

Чтож-уж, если у не работает - приходится экспереминтировать

короче у вас кэп, а у меня веселье

[CalvinKlein]

Вот так будет работать

var el=document.getElementsByClassName('forum');
        for(var i=0; i<el.length; i++) {
        console.log(el[i]);
        el[i].addEventListener('click', function(){gp('viewtopic.php?view='+this.getAttribute('to'), 'GET', '');});
        }

[Ratnikov310]

попрубую

не работает

[hedgehog]

То, что скрипт подключается в самом конце страницы, еще не означает, что динамически создаваемые элементы уже присутствуют на странице. Как минимум нужно время на получение данных. Кроме того, ваши запросы, наверное, выполняются после полной загрузки документа со всеми скриптами

Т.е. вешать события нужно ПОСЛЕ того, как успешно получили данные и сгенерировали контент. Либо на родительские элементы, если позволяет структура (например, li в вашем случае). Как вариант - jQuery on/live

[nerv]

Что я сделал не так

делегирование событий

[CalvinKlein]

попрубую

не работает

если не работает значит к моменту выполнения скрипта тегов с классом "forum" ещё нет

[moron]

Cталкивался с подобной проблемой. Решаемо с помощью делегирования или замыкания.

http://javascript.ru/basic/closure - последний пункт вам ничего не напоминает?