Разное

[iatsdotfatr]

Папку dist лучше убрать в .gitignore, собирать ее и разворачивать на другой ветке, например, через gh-pages. В readme сделать одну ссылку на страницу, с которой можно будет переходить к страницам проекта.

Ознакомиться с правилами https://github.com/fullstack-development/front-end-best-practices, внести соответствующие правки по всему проекту.

[iatsdotfatr]

1. node_modules, несмотря на то, что прописаны в .gitignore, отслеживается, надо так же убрать.
2. Через npm установи модуль gh-pages, который будет папку dist разворачивать на ветке gh-pages, добавь для него скрипт в package.json и укажи в readme.
3. Удалить лишние файлы, вроде report...json из корневой папки, gitkeep из fonts, проверить по всему проекту.
4. Закомментированный код, если не нужен, удалить, если нужен, раскомментировать, проверить по всему проекту.
5. Добавь в requireAll импорт scss-файлы, подобные js-файлы удали. В результате может появиться проблема с scss-ресурсами (переменными, миксинами...), решается либо дополнительными настройками sass-loader, либо установкой и настройкой sass-resources-loader.

[iatsdotfatr]

1. И еще как минимум в пяти местах.
2. Импорты должны быть в самом верху файла или сразу под extends.
3. var в pug заменить на const, если необходимо, то на let.

[iatsdotfatr]

1. Изменения лучше делать в dev-режиме.
2. Scss файлы привести к одному формату (т.е. убрать пустые строки в блоках)

[iatsdotfatr]

1. Использовать двойные кавычки во всех pug-файлах, не только здесь
2. Сокращения не использовать, даже если они кажутся очевидными
3. Не совпадают имена, не только здесь
4. То же самое, лучше оставить один js-файл с инициализацией, остальные вынести в отдельные модули, с donut-chart лучше сделать так же, но это как рекомендация.
5. Лучше еще раз деструктурировать, только не с помощью вложенной деструктуризации, то есть в параметрах миксина, а в теле миксина константой, тоже просто рекомендация

[iatsdotfatr]

1. Свойство существует без префикса, остальное autoprefixer сделает
2. Отрицательные margin запрещены правилами, убрать из всего проекта

[iatsdotfatr]

1. Я не это имел ввиду, говоря про исключение), верни инициализацию в файл с объявлением.
2. Пустые линии убрать, есть не только в это файле
3. Фиксированную высоту убрать, есть в разных местах.

[iatsdotfatr]

1. Убрать пустые блоки
2. Important

[iatsdotfatr]

1. Именование картинок
2. Динамика, перенос делать, только когда в верхней строке места не будет

[iatsdotfatr]

1. 

2. Здесь точно необходим повышенный приоритет?
3. Объединить правила

[iatsdotfatr]

Инлайновые стили использовать только в случае их динамического изменения.

[iatsdotfatr]

Скрипты подключать в нижней части body.

[iatsdotfatr]

Здесь тоже надо поправить в соответствии с БЭМ

[iatsdotfatr]

1. Item тоже деструктурируй
2. В этом проекте тоже в конструкторе оставь только init.
3. Верхнее правило лучше убрать в нижнее
4. Если плагин создает разметку внутри пользовательского элемента, вместо того, чтобы много раз писать important, просто помести это все внутрь класса обертки. Я думал, что здесь проблемы с приорететами, не решаемые посредством css). В других местах, если есть такое, тоже посмотри.

[iatsdotfatr]

1. Позиционирование блока
2. Добавить пустую строку
3. Убрать пустую строку

[iatsdotfatr]

1. Unimportant important.
2. Ссылка не меняет цвет при наведении.
3. ?
4. Клик работает не во всей области.

[iatsdotfatr]

1. Убрать комментарии из webpack и postcss конфигов.
2. Поскольку репозиторий не планируется публиковать, в package.json можно добавить поле private: true, и оставить только поля scripts, dependencies, devDependencies, и, если надо, browserslist.
3. Дропдаун закрывается при нажатии на этот margin.
4. Также у дропдауна тип кнопки "применить" submit, поэтому форма в карте брониравания не будет работать правильно.
5. Календарь открывается при нажатии на заголовки.
6. Здесь получаются два булевых модификатора, влияющих на одно и то же состояние. Должен быть один.