shadowflade / hotel Goto Github PK

Зависимости, которые попадают в конечный исходный код должны быть прописаны в dependencies, а те которые нужны для разработки в devDependencies. Сейчас у тебя в зависимостях есть даже vue-loader)

Посмотри проект, в списках не везде используются семантичные теги, например тут должен список

Или тут

все классы, которые используются для поиска элементов, должны начинаться с префикса js-

Мне не нравится все равно что в папке pug лежит папка common, а в ней лежат глобальные стили, и нет пага самого. Предлагаю в целом избавиться от папки pug. Тк в целом внутри структура неплохая.

Доставай пропсы из объекта, чтобы не обращаться по ключу, так сразу будет видно какие пропсы есть у компонента

футер - это одна сущность. Мини футер нужно сделать для мобильных устройств, с помощью медиа-запросов.

Блоки должны состоять из миксинов и не вызваться в самом блоке, а на странице где этот блок нужен. Сейчас у тебя есть блоки которые через include выполняются, какие то через миксины реализованы, но вызываются в самом блоке.

Советую поставить тебе code spell checker для проверки на орфографию. По дефолту он идет с англ словарем, можно добавить еще и русский язык. И поправь его ошибки пожалуйста. На будущее, можешь сделать это сразу и на слайдере)

1. Блоки должны быть универсальными и не влиять на другие блоки, у тебя сейчас в одном блоке задаются стили для другого, такого быть не должно

2. Миксы у нас запрещены в стандартах

3. Классы не везде семантичные, называть класс по типу first-ul, secondary-ul - неправильно

4. от блока Buttons не вижу никакого смысла, он нигде переиспользоваться не может. Суть блока в том, что он может быть переиспользован, пройдись по проекту, посмотри какие блоки можно, а какие нет смысла где-то использовать еще кроме текущего места.

Привет, я твой ревьюер! чтобы связаться со мной можешь писать мне в телеграмм.

Не очень ясна структура папок.

1. в pug запихано абсолютно все по проекту, даже подключаемые библиотеки, хотя им явное место в папке libs. Сразу по ним замечание. Зачем было выкачивать абсолютно всю ветку этих либ? если тебе по сути нужен только исполняемый скрипт.
2. Также непонятно зачем создавать папку includes а в ней modules. Лучше просто наименовать её components, или modules.
3. layout выглядит как свалка всего чему не нашел место. И переменным и миксинам и тд. В layout должен лежать pug и scss (и если нужно то js) для этого шаблона. Если их несколько - нужно раздробить по папкам. А все глобальное вынести в другое место.
4. Также непонятно зачем в папке pug utils лежит. Эти utils состоят из js файла, к pug они не имеют никакого отношения.

расставь концы строк (;)

Настрой линтер, чтобы вручную это не править

• По макету тут не должно быть жирного шрифта.
  
• На всех кликабельных элементах должен быть cursor: pointer
• в пагинации цифры какие-то слишком маленькие, мне кажется на макете они больше. Так же видно, что стрелка не соотносится с макетом. И эффект активного элемента пагинации не тот
  
• Календарь тоже не тот
  

Проверь еще раз пожалуйста на соответствие макету, имеются явные ошибки. Чтобы не тратить своё и моё время) Так процесс ревью будет гораздо быстрее

• не могу найти, где шрифты лежат?
• именование файлов, блоков, миксинов в разных кейсах где-то camelCase, где-то lower-case, где-то вообще не понятный. Нужно все привести к одному формату 'lower-case'.
• А вот именование переменных должно быть наоборот только в camelCase Потому что по сути это js переменные и им соответствуют правила для js

• с большой буквы называют только классы

не собирается проект

Все что можно сделать с помощью css - нужно сделать через css :) Вот без таких конструкций вспомогательных