Почему не forEach? Меньше писанины и выглядит лучше. Регистрировать переменную target, если потом нигде её не используешь. var attr = target.dataset.attr.dataset.attr; и везде так..ю

[f0cusX]

fetchJS/js/main.js

Line 94 in 2ddaeac

for (var i = 0; i < button.length; i++) {