Неправильные символы комментирования в css-файлах Принято

1

Уважаемые разработчики фреймворка и приложений!
Пожалуйста, обратите внимание на то, что в *.css-файлах правильным способом закомментировать что-то является конструкция /* комментарий */

У вас же во множестве *.css файлов имеются комментарии вида // коментарий. В результате чего, такие IDE как NetBeans и PhpStorm считают эти строки ошибками.
И это верно. Во всех спецификациях CSS указано что комментарии в CSS могут быть только /* такие */. Например: https://developer.mozilla.org/en-US/docs/Web/CSS/C...

Если просто запустить по всем приложениям поиск CSS-файлов со строками вида // - можно увидеть что такое встречается регулярно.
Примеры: (на момент написания версия webasyst 1.5.13.56, shop-script 7.1.4.68), реально таких файлов десятки.

wa-content/css/wa/wa-1.0.css

ul.menu-h li { display: inline-block; //display: inline;
ul.tabs li { display: inline-block; //display: inline;
ul.toggle li { display:inline-block; //display:inline;

wa-apps/blog/themes/default/default.css

border:1px solid #bbb; //border:none

wa-content/css/jquery-wa/wa.ui.editor.css

.ui-editor-toolbar .ui-buttonset, .ui-editor-toolbar .ui-button { //display:inline!important; width:auto; zoom:0;}

Вот это последнее даже затрудняюсь представить себе как будет распознано браузером. Тут явно ваш минификатор приложил руку, уже после того как кто-то добавил // в одну строку как ему показалось.

Просьба принять меры в виде настрйоки css-анализаторов, которые бы выдавали ошибки при тестировании и сборке, и не давали выпускать такие файлы.

Такие комментарии во-первых могут вызывать непредсказуемые ошибки при разборе их браузерами, а во-вторых смотрятся просто непрофессионально.

Спасибо.


2 комментария

  • +2
    golubevmark golubevmark Webasyst 31 октября 2016 10:28 #

    В "wa-1.0.css" подобные конструкции использовались во времена царствования и величия браузера IE7. Как известно "inline-block" начал поддерживаться только с 8й версии, а для 7й использовался подобный трюк с "dispay: inline" и т.д. Код ничего ломать не должен.

  • +1
    Михаил Рубин Михаил Рубин 31 октября 2016 18:37 #

    Спасибо за ответ. А вы всё ещё поддерживаете IE7?

    Если нет, возможно стоит хотя-бы не добавлять новые подобные конструкции?

    Добавить комментарий

    Чтобы добавить комментарий, зарегистрируйтесь или войдите