metanonum
commented
2 years ago - Pug поддерживает новые возможности js, не забывай про них.
var->const'colors__square_' + colorItem-> `color-section__squarecolor${color}` Здесь еще не очень удачное именование. Это блок для одного конкретного цвета, почему он во множественном числе? И ошибка БЭМ: неверное именование модификаторов. Здесь множество значение для одного типа модификатора, т.е. именование должно быть ключ_значение. - Стиль отступов должен быть единообразным.
Такого что где-то 2 пробела, а где-то 4 быть не должно. А произвольное количество тем более. Настройка отступов по-умолчанию в редакторе и автоисправления притиера могу упростить жизнь.
И не забывай еще eslint и stylelint проверять. Вероятно у тебя не подсвечиваются ошибки в .scss. Для vscode есть соответствующие плагины читающие твою конфигурацию и проверяющие код сразу в редакторе. Для других ide есть подобные плагины. + функционал притиера для автоисправления по горячим клавишам или в контекстном меню. - Нужно ограничить длину строки до 70-80 столбцов, я в правках для js упоминал, что к
.pugэто тоже относится.
- У всех элементов форм, которые передают информацию, должен быть
name, уникальный в пределах этой формы. - Поле Email не должно быть типа
text. HTML 2.17
<form>HTML 2.14