mvodev / hotel Goto Github PK
View Code? Open in Web Editor NEWFSDCourse2 layout
FSDCourse2 layout
Сложно ориентироваться в не совсем стандартной структуре директорий.
Понятней будет, если поместить блоки БЭМ (те, что сейчас в src/ui-kit/modules
) в components
, а страницы - в pages
:
/src
/components # блоки БЭМ
/pages # страницы
Не ясно какую смысловую нагрузку несет в себе то разделение изображений опираясь на расширение, возможно присмотревшись внимательно ты придумаешь как их грамотно распределить, к тому же не все изображения должны храниться в assets, если изображение строго привязано к кому-то компоненту, то есть смысл положить его в том же компоненте.
https://github.com/fullstack-development/front-end-best-practices/blob/master/Initiation.md#2
Лучше везде делать в два пробела, табы не годятся, в один пробел - тоже.
Тут тоже вопрос к единообразию, кавычки вокруг значений атрибутов отличаются, когда будешь писать на реакте используя синтаксис jsx, можешь по привычке пренебрегать стандартизацией, а затем правило для кавычек тимлид может описать в eslint и вылезет куча предупреждений мелких.
В данном файле указанно, что изображения лежат в корне проекта, но в продакшен билде фавиконок в корне нет, пути нужно поправить, это касается и browserconfig.xml. Как я понимаю ты сгенерировал фавиконки на генераторе, он по умолчанию генерирует их так, будто они будут лежать в корне проекта, а если фавиконки некоторые типов лежат в корне проекта их не нужно импортировать, учти это и перепроверь пожалуйста, что подключено, а что нет )
В продакшен билде, шрифты лежат и в корне проекта и в assets.
jquery
) должны быть в dependencies
.typescript
и ts-loader
лишние, если проект написан полностью на js?На строке 19 ошибка в пути @import '../components/heading//heading.scss';
.
Предпочтительно перенести импорт стилей в те компоненты куда подключается остальная часть компонента. Например ты подключаешь компонент button в компоненты dropdown, header-bar и т.д., стили компонента button нужно подключать в этих компонентах.
Перенеси пожалуйста в dependencies всё то, без чего твоя продакшен сборка не соберется или не будет работать после компиляции, а в devDependencies остальное. Это один из вариантов распределения зависимостей, будет полезно ознакомиться.
Класс-модификатор не может использоваться без основного класса блока/элемента:
Должно быть: class="form-elements__buttons-wrapper form-elements__buttons-wrapper_first-row"
К тому же хорошие практики запрещают использование примесей. Их следует заменить на модификаторы, например: class="button button_variant_with-arrow
.
По заданию макс ширина 1920, но лучше делать чтобы на 1920+ тоже все выглядело хорошо. То есть растянуть по ширине фоновое изображение, продлить футер и хедер. При этом сам контент оставить где он есть.
Футер должен быть прижат к низу https://github.com/fullstack-development/front-end-best-practices/blob/master/CSS/README.md#1.15
Количество младенцев нужно считать тоже:
Пример:
Сюда нужно запретить писать:
Здесь кнопка, как будто не относится к инпуту, когда он в фокусе:
Здесь текст явно не вровень с маркером:
В задании есть требование pixelperfect. Просьба проверить, что все совпадет.
Курсор должен меняться на pointer, когда на ползунок наводим (и на все интерактивные элементы):
К скриптам такое же требование как и к стилям тут.
var
в файлах pug, следует заменить на const
или let
. На JavaScript в файлах pug также распространяются best practices.button
перегружен, лучше разбить на несколько, по одному на каждый блок (button
, like-button
, etc). В целом следует избегать слишком "универсальных" компонентов - поддерживать код будет гораздо проще.checked=isChecked
.Добавить папку docs в .gitignore.
Чтобы не писать ишью по каждому пункту, прошу перечитать и поправить код
https://github.com/fullstack-development/front-end-best-practices
Страницы не проходят валидацию.
A declarative, efficient, and flexible JavaScript library for building user interfaces.
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google ❤️ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.