Формирование JavaScript файлов и кусков кода для дополнений

Всех приветствую. Есть ситуация, с которой сталкиваюсь на каждом сайте, к которому заказчик предъявляет требования по части валидности верстки\кода. Этот пост про ситуацию с кусками java-скриптов, которые подключают в код страниц дополнения modx. Например, AjaxForm, MinifyX и MiniShop2 подтягивает в код страницы блоки javascript которые открываются тегами:

<script type="text/javascript">

И этот тег не проходит валидатор validator.w3.org
Валидатор выдает такое предупреждение:
The type attribute is unnecessary for JavaScript resources.
Чтобы исправить эту ситуацию — нужно убрать из тега атрибут type=«text/javascript».

Господа разработчики, постарайтесь обновить свои дополнения, чтобы сделать код сайтов на modx более валидным.

Заранее благодарен!

UPD.: Решение проблемы здесь bazstudio.com/blog/320-make-valid-the-auto-js-files-in-modx

Wassi Wassinen

24 ноября 2018, 18:58

4 376

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

Михаил

24 ноября 2018, 21:28

Какая оплата?

Илья Уткин

24 ноября 2018, 21:35

Насколько я знаю, это сообщение просто подсказывает, что аттрибут type неважен (необязателен), если вы вставляете блок javascript-кода. Нигде нет указания, что этот аттрибут запрещён или нерекомендован.

Wassi Wassinen

24 ноября 2018, 21:48

Я с вами абсолютно согласен. Но поисковикам и заказчикам нет особого дела до нюансов. Есть тип “warning” и “error” которых не должно быть на сайтах.

Илья Уткин

24 ноября 2018, 21:50

Насчёт заказчиков согласен, что действительно есть упоротые на некоторых нюансах. А насчет поисковиков — не согласен)

Wassi Wassinen

24 ноября 2018, 21:52

Про поисковики отчасти уступлю. Но основной «затык» в заказчиках, их регламентах и т.п. :)

Stanislavsky

04 декабря 2018, 11:49

Читал как-то в новостях по front-end разработке, что гугл будет ругаться на указание типа text/javascript. Теперь достаточно указывать тег script без этого атрибута. Эта информация была еще в октябре. Сейчас искал статью, но найти не могу, к сожалению.

Евгений Webinmd

25 ноября 2018, 00:13

так вроде не разработчики указывают type=«text/javascript», большинство использует код

$modx->regClientStartupScript

или что-то такое

А там указано

$this->sjscripts[count($this->sjscripts)]= '<script type="text/javascript" src="' . $src . '"></script>';

Пруф

Wassi Wassinen

25 ноября 2018, 01:18

Евгений, спасибо за ответ! Это можно как-то исправить не меняя исходных файлов?

Евгений Webinmd

25 ноября 2018, 13:51

только если отправить PR на гитхабе MODX и если его примут))

Василий Столейков

25 ноября 2018, 01:19

Тоже постоянно сталкивался с такой же проблемой.
Решил для себя проблему простейшим плагином и написал инструкцию к этому: https://bazstudio.com/blog/320-make-valid-the-auto-js-files-in-modx

Wassi Wassinen

25 ноября 2018, 01:21

Спасибо огромное!

Wassi Wassinen

25 ноября 2018, 01:25

Идея классная, но у меня не сработало. :)
Modx 2.6.5

Василий Столейков

25 ноября 2018, 01:30

аааа… У меня почему-то съедало код, а точнее строку, которую нужно удалить. Обновил статью, проверь, там в комментариях про пробел написал пока что.

Wassi Wassinen

25 ноября 2018, 01:32

Да, всё супер. Вы молодец! Еще раз благодарю :)

Wassi Wassinen

25 ноября 2018, 01:29

Прошу прощения. «Скопипастил» и не увидел, что нужно добавить нужное для вырезания значение. :) Было бы понятнее для новичков, если прямо в примере указать type=«text/javascript» в кавычках. :)

Авторизуйтесь или зарегистрируйтесь, чтобы оставлять комментарии.

Формирование JavaScript файлов и кусков кода для дополнений

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

Здесь упомянуты