1

Нужно из переданного (по id) блока найти все картинки и каждой присвоить уникальное значение атрибута.

Сейчас сделано так:

$("#detailArt").find("a:has(img)").attr('data-fancybox', Math.random());

Но, к сожалению, данная конструкция присваивает всем картинкам одинаковое число, то есть обращается к функции Math.random() только один раз для всех найденных элементов.
Как это исправить? Нужны уникальные значения для каждого элемента.

2 ответа 2

2

Попробуйте так:

$("#detailArt").find("a:has(img)").each(function() {
   $(this).attr('data-fancybox', Math.random());
});
2
  • Спасибо, помогло. 13 июл 2018 в 11:34
  • ну и добавить вместо Math.random(): 'img_' + Math.round(Math.random() * Math.pow(10,10))
    – diproart
    13 июл 2018 в 12:19
1

Один из вариантов передаваемых параметров в функцию attr - это название атрибута и функция, возвращающая значение данного атрибута для конкретного элемента

При использовании такого варианта получается так:

$("#detailArt").find("a:has(img)").attr("data-fancybox", function() { return Math.random(); });

В более современном оформлении:

$("#detailArt").find("a:has(img)").attr("data-fancybox", () => Math.random());

И, наконец, вариант не очень правильный*, но ещё более короткий:

$("#detailArt").find("a:has(img)").attr("data-fancybox", Math.random);

* По-хорошему, в Math.random не должны передаваться параметры

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.