TimurPrh / toxin-hotel

MetaLamp toxin hotel task
0 stars 0 forks source link

Структура #4

Open daswetter73 opened 2 years ago

daswetter73 commented 2 years ago
  1. Медиа запросы должны находится внутри класса image
  2. Весь JS для блоков должен быть модульным. И наилучшее решение для такого подхода - классы.
  3. Использовать возможности препроцессора по максимуму, пройтись по всем файлам и где можно применить инструменты pug, к примеру, сделать это. image
TimurPrh commented 2 years ago

Исправил медиа запросы и классы js https://github.com/TimurPrh/toxin-hotel/commit/f8ed3c5d0731d9bff6809cf267bf96c1b0d08173 (забыл в сообщении добавить #4)

daswetter73 commented 2 years ago
  1. Лучше задавать имена переменным, которые не зависят от конкретного цвета, к примеру primary или secondary image
  2. Библиотеки должны быть установлены через npm image
  3. Применить возможности pug (пройтись по всему проекту!) image image
  4. Название элемента лучше писать вместе, к примеру, footer-mobile, но лучше вообще переименовать этот класс, тк переводится как 'футер-мобилка' image
  5. Нужно сделать код читаемым image
  6. js файл блока должен называться также, как и класс, а инициализация должна происходить в другом файле, к примеру, можно назвать его init
daswetter73 commented 2 years ago
  1. Название элемента лучше писать вместе (2). Просмотреть код и исправить везде image
  2. Здесь есть повторяющиеся части image image image
  3. Если text-types.scss не используется, то лучше его удалить
  4. Аналогичная ошибка, как и 7 в том же файле image
daswetter73 commented 2 years ago
  1. Медиа запросы должны находится внутри класса image
  2. Нет смысла повторять свойства и переменные, если они одинаковые. Необходимо добавить дефолтные значения image
  3. Избегать использования !important image
  4. Вынесли все include в начало файла
  5. Добавить и настроить линтеры для scss и pug, чтобы код выглядел единообразно