sohtr / toxinhoteluikit Goto Github PK

Сделай так, чтобы у тебя в точке входа была бы функция, которая реквайрила твои стилевые файлы, файлы js, файлы картинок, чтобы не приходилось помнить о каждом модуле и подключать его вручную. Заодно исчезнут директивы @import в стилевых файлах. И тогда твое подключение блоков сведется к указанию include внутри pug-файлов, а остальное будет собираться само.

Нарушено требование почти во всех js-ках компонент
https://rizzoma.com/topic/a87a0a28b84d6326d4f3909e8801dab7/0_b_8nel_88gv1/

Добавить команду для клонирования репозитория

Тут использование массивов для хранения данных само собою напрашивается)

Зачем сейчас в корне и в папке src лежат файлы index.html? Зачем в src лежат беспризорные js файлы, которые не относятся к блокам? Зачем два файла index.js один в корне, один в src? Такой файл должен быть один в src и все импорты должны быть в нем. На тему импортов еще будет отдельное замечание.
Никакой отдельной папки js в src быть не должно. Как и никакой отдельной папки scss там же. Все это являются частью реализации блоков и должно лежать в папках блоков (а так же картинки относящиеся к блокам и прочие файлы если таковые есть). Исключение папка src - здесь маожет лежать один корневой pug файл, один js, как точка входа для webpack и один scss файл в котором описано то что относится к index.pug. Тут же в src может быть папка images в которой лежат либо картинки для всего проекта общие (конкретные картинки блоков лежат в самих папках блоков), либо фавиконки. Файл nano .babelrc должен лежать в корне проекта рядом с конфигом webpack.

Каждый проверяющий (а в последствии и работающий с твоим репозиторием при командной разработке) должен клонировать репозиторий, установить зависимости с помощью npm i и все скрипты должны работать. Сейчас при запуске сервера выдает следующую картину:

при сборке dev тоже выдает ошибки. На сколько я понимаю что-то со шрифтами.

используй rem в проекте

Что это за именования?)))

Добавить описание команд запуска скриптов сборки, сервера, установки пакетов зависимостей. Добавить явное описание команды для клонирования репозитория. Добавить ссылку на GitHub Pages. Любой проверяющий/использующий проект должен такую информацию видеть в описании, а не смотреть по кишкам файлов, а что же там надо запустить чтобы собрать проект и так далее.

Разнести зависимости для devDependencies и dependencies

Пробегись по всему проекту, и удали дублирование кода

Не используй внешнее позиционирование блока

Лучше не ограничивать высоту контента, пусть лучше ее определяют дочерние элементы

Нарушено требование почти во всех js-ках компонент
https://rizzoma.com/topic/a87a0a28b84d6326d4f3909e8801dab7/0_b_8nel_78v67/

Крашется верстка под мобильную версию

Все файлы проекта именовать в соответствии со стандартами FSD.

1. В проекте полностью соблюдать требование методологии БЭМ. Исключение - позиционирование блоков, вместо миксов их нужно помещать в обертки, которые являются элементами родительского блока по БЭМ. И уже на эти обертки (элементы) вешать позиционирование. Миксы не допустимы.
2. Блоки по БЭМ должны быть реализованы миксинами pug. Параметры блоков должны передаваться объектами js - в корневом блоке задаваться все параметры, внутри распределятся по более мелким блокам. То есть у проекта должна быть древовидная структура. Это требование компонентности верстки подобно подходу, применяемому в react.

1. Размеры в соответствии со стандартами должны быть по большей части в единицах rem.
2. Шрифты нужно подключать в теге body и они потом наследуются, смысла подключать их в каждом блоке, если они не меняются - нет.
3. Кроме того сами шрифты подключаются не совсем так как у тебя сейчас. Там в файле fonts.scss где у тебя шрифты, одному и тому же семейству дается одно и то же название, но в разных font-face ты еще и указываешь начертание и жирность текста. Это потом для одного и того же шрифта служит триггером какие файлы шрифта использовать. То есть например здесь

a{
  margin: 0px;
  padding: 0px;
  font-family: 'Montserrat';
  font-size: 0.875rem;
  color: rgba(31, 32, 65, 0.5);
  list-style: none;
  text-decoration: none;
  outline: none;
}
p{
  margin: 0;
}
h1{
  font-size: 24px;
  line-height: 30px;
}
h2{
  font-size: 19px;
  line-height: 24px;
}
h3{
  font-size: 12px;
  line-height: 15px;
  font-family: 'MontserratBold';
  font-weight: bold;
  text-transform: uppercase;
  color: #1F2041;
  opacity: 0.75;

ты указываешь не font-family разный, а указываешь просто font-weight: bold или font-weight:normal и так далее, а font-family где-нибудь один раз в body.

1. Теги не должны быть без классов.
   Пример p Итого это во всем проекте по аналогии.