Формирование 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
modx.pro
6
3 557
+5

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

Михаил
24 ноября 2018, 21:28
-1
Какая оплата?
    Илья Уткин
    24 ноября 2018, 21:35
    0
    Насколько я знаю, это сообщение просто подсказывает, что аттрибут type неважен (необязателен), если вы вставляете блок javascript-кода. Нигде нет указания, что этот аттрибут запрещён или нерекомендован.
      Wassi Wassinen
      24 ноября 2018, 21:48
      0
      Я с вами абсолютно согласен. Но поисковикам и заказчикам нет особого дела до нюансов. Есть тип “warning” и “error” которых не должно быть на сайтах.
        Илья Уткин
        24 ноября 2018, 21:50
        +1
        Насчёт заказчиков согласен, что действительно есть упоротые на некоторых нюансах. А насчет поисковиков — не согласен)
          Wassi Wassinen
          24 ноября 2018, 21:52
          0
          Про поисковики отчасти уступлю. Но основной «затык» в заказчиках, их регламентах и т.п. :)
        Stanislavsky
        04 декабря 2018, 11:49
        0
        Читал как-то в новостях по front-end разработке, что гугл будет ругаться на указание типа text/javascript. Теперь достаточно указывать тег script без этого атрибута. Эта информация была еще в октябре. Сейчас искал статью, но найти не могу, к сожалению.
        Евгений Webinmd
        25 ноября 2018, 00:13
        0
        так вроде не разработчики указывают type=«text/javascript», большинство использует код

        $modx->regClientStartupScript
        или что-то такое

        А там указано

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

        Пруф
          Wassi Wassinen
          25 ноября 2018, 01:18
          0
          Евгений, спасибо за ответ! Это можно как-то исправить не меняя исходных файлов?
            Евгений Webinmd
            25 ноября 2018, 13:51
            0
            только если отправить PR на гитхабе MODX и если его примут))
          Василий Столейков
          25 ноября 2018, 01:19
          +4
          Тоже постоянно сталкивался с такой же проблемой.
          Решил для себя проблему простейшим плагином и написал инструкцию к этому: https://bazstudio.com/blog/320-make-valid-the-auto-js-files-in-modx
            Wassi Wassinen
            25 ноября 2018, 01:21
            0
            Спасибо огромное!
              Wassi Wassinen
              25 ноября 2018, 01:25
              0
              Идея классная, но у меня не сработало. :)
              Modx 2.6.5
                Василий Столейков
                25 ноября 2018, 01:30
                0
                аааа… У меня почему-то съедало код, а точнее строку, которую нужно удалить. Обновил статью, проверь, там в комментариях про пробел написал пока что.
                  Wassi Wassinen
                  25 ноября 2018, 01:32
                  0
                  Да, всё супер. Вы молодец! Еще раз благодарю :)
                Wassi Wassinen
                25 ноября 2018, 01:29
                +1
                Прошу прощения. «Скопипастил» и не увидел, что нужно добавить нужное для вырезания значение. :) Было бы понятнее для новичков, если прямо в примере указать type=«text/javascript» в кавычках. :)
                Авторизуйтесь или зарегистрируйтесь, чтобы оставлять комментарии.
                15