Мелкие замечания по pug

[imin314]

В arrow-button у тега не задан атрибут type, а, значит, кнопка по умолчанию будет иметь тип 'submit'. Этот тип используется для отправки формы на сервер, но парные кнопки-стрелки вряд ли предназначены для этого.

Зачем нужно разделять название файла от пути к нему? Переменная avatarName нигде, кроме как, пути не используется.

[Ilanevazno]

Поправил подключения, поправил arrow button Случайно запушил с ишью #34

[imin314]

1. Если в миксин передаются параметры по умолчанию, их не нужно явно указывать. Также можно для текста вместо дефолтного значения text указать 'Button', тогда не придется и его передавать в миксин.

2. Между include и mixin надо добавить отступ, по аналогии со стилем оформления импортов в js:

3. Для перебора массива параграфов лучше использовать цикл each, как ты делаешь в других файлах:

4. Объект logo используется только для того, чтобы получить из него свойство link. Лучше просто создавать переменную logoLink со строкой.

[Ilanevazno]

Поправил по всем файлам

[imin314]

1. Не хватает отступов:

2. Надо добавить запятые после последнего свойства объекта (стандарты airbnb):

3. each - это встроенный оператор pug. Перед ним не нужен дефис:

4. Для обращения к свойствам вложенных объектов тоже лучше использовать деструктуризацию:

5. Не хватает пробелов в фигурных скобках:

6. Здесь использование let для theme и size кажется неоправданным. В theme и size хранятся ключи модификаторов, а после в коде ты присваиваешь им название модификатора целиком. Лучше theme и size сделать const, а ниже объявить новые переменные sizeModifier и themeModifier (тоже через const, т. к. применение модификатора -reverse можно объединить с объявлением переменной).

[Ilanevazno]

Исправил.

[imin314]

1. Надо добавить еще отступ перед закрывающей фигурной скобкой для объекта с параметрами (14 строка).

И несколько новых:

1. search-dropdown-section: В массивах не нужны пробелы после открывающей [ и перед закрывающей ]. Запятая в однострочном массиве после последнего элемента не ставится. Лишняя запятая:

2. stages: Не используй var. Перед if/else можно удалить дефисы. Для отображения номера используется css, но по стандартам компании нужно, чтобы тексты по максимуму отображались даже без css и html (пункт 1).

3. stats-section.pug: +percentages-chart({ currentFill: '98', }) - лищняя запятая

4. svg-icon.pug: svg xmlns="http://www.w3.org/2000/svg" (svg - тег по ошибке стал атрибутом)

5. toggles.pug: section.toggles и label.toggles__label оказались на одном уровне вложенности.

6. В contacts.pug надо генерить элементы .contacts__contact в цикле.

[Ilanevazno]

Надо добавить еще отступ перед закрывающей фигурной скобкой для объекта с параметрами (14 строка). Не совсем понял, возможно я уже это поправил во время рефакторинга, но не нашёл где нужно добавлять пробел.

2. Уже правлено во время рефакторинга
3. Заменил var на let, и не совсем понял, зачем в данном виджете подгружать текст через html, ведь тогда получится странный и бесполезный список, но стандарты есть стандарты ))
4. лишнюю запятую убрал
5. Поправил svg
6. Поправил вложенность
7. Отрефакторил контакты, циклом выглядит правильнее.)

[imin314]

1. Этот пункт был про news-events-section:

2. Разметка в if и в else отличается только наличием класса. Лучше не дублировать код в if/else, а динамически определять класс.

   • вместо того чтобы считать counter + 1, можно начать цикл с 1, а закончить, когда счетчик будет равен maxStep:

3. Здесь не нужно деструктурировать объект contact, т. к. в миксин contact-section ты передаешь тот же объект. После миксина зачем-то = contact.

[Ilanevazno]

1. добавил отступ;
2. поправил исправил 'counter +1' и дублирование кода в if/else
3. поправил, так выглядит намного лучше