Почему не работает forEach в ie10+, edge14+ ??

Question

[BB 8]

Где можно почитать более детаельно о кроссбраузерности JS ??

Comments

[Дмитрий Лузанов]

Что значит

не работает forEach

?

[0xD34F]

А можно взглянуть на пример неработающего кода? - работать должно. Наверняка не в forEach проблема.

[Павел Корнилов]

Работать должен, проверено многократно. Нужен пример неработающего кода, можно в упрощенном виде.

[BB 8]

Predve4niy:

test.pie.zzz.com.ua

не работает параллакс в шапке + не работают кнопки в секции schedule(в остальных секциях тоже не работают кнопки). В пятницу смотрел в IE11 и EDGE14.

[BB 8]

0xD34F:

test.pie.zzz.com.ua

не работает параллакс в шапке + не работают кнопки в секции schedule(в остальных секциях тоже не работают кнопки). В пятницу смотрел в IE11 и EDGE14.

[Дмитрий Лузанов]

не работают кнопки в секции schedule(в остальных секциях тоже не работают кнопки).

Так кнопки или forEach?

[Дмитрий Лузанов]

Приведите пример кода, который не работает.

[BB 8]

Predve4niy:

tabs.forEach(function (el, i) {
	 el.addEventListener('click', function (e) {
		var currentTabIndex = i;

		tabs.forEach(function (el, i) {
			el.classList.remove('o_active');
			innerTabs.forEach(function (el, i) {
				el.classList.remove('tabs-active')
				if (i === currentTabIndex) {
					el.classList.add('tabs-active');
				}
			})
		});
		el.classList.add('o_active');
	 })
})

[0xD34F]

BB 8: forEach применяется к NodeList. Ниже уже ответили.

[Дмитрий Лузанов]

BB 8:

var forEach = function(collection, cb) {
  return Array.prototype.forEach.call(collection, cb)
}

forEach(tabs, function (el, i) {
   el.addEventListener('click', function (e) {
    var currentTabIndex = i;

    //--------------
   })
})

[Дмитрий Лузанов]

BB 8:
А зачем вы два раза берете форыч над одной и той же коллекцией?

tabs.forEach(function (el, i) {
   el.addEventListener('click', function (e) {
    var currentTabIndex = i;

    tabs.forEach(function (el, i) {

[BB 8]

Predve4niy: затем, что сначала действия выполняются для самой коллекции элементов, а в следующей, проверяется соответствие индексов с другой коллекцией и, если они совпадают, происходят действия описанные в коде. Это табы написанные собственноручно :D Или это неправильный подход ?)

Answer 1

[dom1n1k]

Подозреваю, что автор пытается применить forEach к коллекции типа NodeList - а там поддержка forEach появился совсем недавно и кроссбраузерность ещё плохая.
В отличие от простых массивов - там всё давно работает.

Comments

[BB 8]

Всё верно. forEach применяется к коллекции NodeList'a.
Как решить проблему кроссбраузерности ?

[0xD34F]

BB 8: Добавить полифилл.

[dom1n1k]

Либо полифил, либо заменить forEach на рабоче-крестьянский for.

Answer 2

[Александр Аксентьев]

var divs = document.querySelectorAll('.app');

[].forEach.call(divs, function(div) {
    
});

Comments

[Vadim Rublev]

А откуда div в аргументе? Или что это такое?

[Vadim Rublev]

аа... Понял.
У меня заработало нормально вот так:
function(item, index, array)
где item (он же div) - обрабатываемый элемент.

Вариант function(item) в "Edge" один раз отрабатывает и подвешивает его.

Answer 3

[Ваге]

добавьте этот полифилл

if (typeof Array.prototype.forEach != "function") {
    Array.prototype.forEach = function(callback) {
        for (var i = 0; i < this.length; i++) {
            callback.apply(this, [this[i], i, this]);
        }
    };
}