Git Product home page Git Product logo

hotel's Introduction

  • 👋 Hi, I’m @ShadowFlade
  • 👀 I’m interested in full-stack development
  • 🌱 I’m currently learning React, Typescript.
  • 💞️ I’m looking to collaborate on any frontend project.
  • 📫 How to reach me [email protected]

You can see my projects at :

https://shadowflade.github.io/slider/ (https://github.com/ShadowFlade/slider)

https://shadowflade.github.io/hotel/ (https://github.com/ShadowFlade/hotel)

https://shadowflade.github.io/google-books/ (https://github.com/ShadowFlade/google-books)

Anurag's GitHub stats

Top Langs

My Skills

hotel's People

Contributors

shadowflade avatar

Watchers

 avatar

hotel's Issues

Именование

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

image

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

image

Footer

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

Зависимости

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

Структура папок

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

Семантика

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

JS-префикс

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

Пропсы

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

БЭМ

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

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

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

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

Css

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

Структура папок

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

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

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

JS

расставь концы строк (;)
image
Настрой линтер, чтобы вручную это не править

UI

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

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

Pug

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

Опечатки

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

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.