minishop2 msGallery default.js alt и title

При смене большого изображения в галерее карточки товара по клику на превьюшку alt и title изменившегося изображения остаются прежними.
Что же делать? Разобрался. Может кому-нибудь, кто озабочен SEO, а также дотошным, пригодится.
Функционал msGallery находится в файле assets/components/minishop2/js/web/default.js
Строку
$(miniShop2.Gallery.mainImage, miniShop2.Gallery.gallery).attr('src', src).parent().attr('href', href);
необходимо заменить на 
var alt = $(this).find('img').attr('alt');
var title = $(this).attr('title');
$(miniShop2.Gallery.mainImage, miniShop2.Gallery.gallery).attr('src', src).attr('title', title).attr('alt', alt).parent().attr('href', href).attr('title', title);
А также вопрос Василию:
Где принимаются предложения по улучшению кода minishop2?
Сергей Малышев
Сергей Малышев
07 июля 2017, 12:57
modx.pro
1 597
0

Комментарии: 5

    Сергей Малышев
    Сергей Малышев
    07 июля 2017, 16:37
    0
    И как добавить свой пул реквест?
      Алексей Ерохин
      Алексей Ерохин
      07 июля 2017, 16:44
      0
      Делаете форк, изменяете файлы, делайте коммит в форкнутом репозитории, потом делаете пул-реквест. Примерно так.
    Сергей Малышев
    Сергей Малышев
    07 июля 2017, 16:40
    0
    Может у Василия где-то об этом написано? Дайте ссылочку, пожалуйста. А то не хочется досконально разбираться в гитхабе.
      Леонид
      Леонид
      25 мая 2019, 21:19
      0
      Приветствую. На данный момент нет такой строчки //$(miniShop2.Gallery.mainImage, miniShop2.Gallery.gallery).attr('src', src).parent().attr('href', href);// в assets/components/minishop2/js/web/default.js.
      Попытка прописать значения для alt непосредственно в tpl.msGallery не принесли результатов, вывод штатно по описанию и названию картинки в самой галерее не происходит,. Есть ли у кого идеи? Яндекс из-за этого половину страниц в дубль отправил.
        Авторизуйтесь или зарегистрируйтесь, чтобы оставлять комментарии.
        5