kinda-neat / ui-kit Goto Github PK

public не коммитят, потому что её контент вычисляется динамически и

1. Нет смысла хранить то, что можно собрать одной командой на основе имеющегося кода
2. Там может быть много файлов, которые будут отжирать место на диске, долго скачиваться и так далее
3. При мёрже веток постоянно будут конфликты

Вместо этого в package.json добавляют команду build, при помощи которой (npm run build) эта папка собирается из исходников

Прошу обратить внимание на messaging, зачем тут два вложенных блока? У нас идеология, что каждый блок — в отдельной папке. так что тут либо разноси по разным компонентам, либо делай это все элементами :)

Просто внутрь блока положи все переопределения селект2 :)

На самом деле необязательное issue. Больше "на подумать", "Задание со звездочкой" и тд.

Попробуй объединить реализацию этих слайдеров в один компонент. Который в зависимости от переданных компонентов будет выдавать нужную реализацию слайдера. А тебя щас два отдельных элемента для каждого слайдера и у них есть общие классы со схожими стилями. Это же дублирование и чем больше слайдеров, тем больше это будет проявляться и тем больше придется копипастить js,pug,less файлы из одной папки в папку для нового слайдера и там все подправлять. Они по этому и названы щас не очень: slider-a slider-b.

PS. Есть еще 1 косячок

Но это ошибка в выбранной библиотеке, наверно, не целесообразно будет ее править.

У тебя получается, что два блока в main.pug: main-page и main-page-content. Можно сделать через один.
И left - right у main-page-content будет модификатором.

Папку с зависимостями проекта не коммитят в гит, вместо этого коммитят package.json с описанием зависимостей. Чтобы не хранить кучу файлов в гите. Часто там очень много файлов, я встречался с node_modules > 1 Гб.

TODAY ( у тебя "ANOTHER DAY") должна быть кнопкой, которая кликается и по клику выставляется текущая дата. Да, это сложно понять по макету, но это так...

Не буду говорить какие правила, но есть явные небрежности

Можешь пояснить зачем нужно

Намешаны табы пробелы, встречаются и одинарные и двойные кавычки, нет пробела после //

Где-то в проекте он есть.

Они же всё равно округляются

Всегда имена должны отражать смысл, а не стилизацию или позицию.

.select2.select2-container.select2-container--default { width: 100% !important; }

Лучше через префикс указывать вид/тип компонента, а не через цифру .

Нужно было попрактиковаться в стилизации дефолтного видео - ты просто вставил ютюбовский через фрейм)

Если что, есть псевдокласс :disabled в css

Все вызовы локализовать в одном методе $(this.domEl).find(".js-calendar__header");

В некоторых местах используется float можно убрать. Это касается и первого макета. Тогда без него можно убрать и clearfix. Особенно если это нужно выставить два блока рядом. В этом случае можно просто менять display с inline-block на block и обратно.

Разве там для каждого стиля необходим !important ?

Ну а вообще сам решай, это мелочь

Теперь в calendar.js

var bottomBlock = $(this).html('Today');
Тут же просто можно вызвать jquery функцию. Создавать переменные, которые не будешь использовать, не надо.

Обязательно проверь, что все правильно настроено, сделай специально какюу-нибудь ошибку именно из правил airbnb и должен тебе линтер подсветить в IDE это и через консоль выдать варнинг. Сейчас у тебя много мелочей из-за которых ощущение, что ты вообще линтером не проходился по проекту :)

Например, шаги signup назвать по смыслу, что там в шагах делается. От цифр в именах файлов, классов и тд надо уходить по максимуму

Папку vendor, опять же, не коммитят, потому же, почему остальные зависимости. Надо использовать те же зависимости, но установить из через npm и получать через require

Если будешь настраивать ESLint, можно начать с официального гайда

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

Не убраны дефолтные outline у инпутов. Этот пункт есть в стандартах.

Сейчас на каждой странице идет как минимум блок самой страницы и внутри блок с постфиксом content. А почему вложенный блок, а не элемент __content? И почему у index.pug целая куча влоденных блоков и это не в рамках всех страниц сделано (не в той же папке)? :)

а не вложенными функциями

Ну, сам понимаешь)

$(this)[0]; — а какой в ней смысл? :)

У тебя по-моему нигде Arial не указан.

В макете со всеми элементами конкретно layout-search-drop-user-profile-container-left__search выпадает за края.

.arrow-button__arrow_disabled_true
Получается, что как будто два модификатора на одном элементе: disabled и true. Также можно использовать дефис в этом случае. И конкретно здесь true можно убрать. Это будет уже достаточно ясно

Все скрипты должны быть оформлены классами в соотвествии вот с этим пунктом стандартов (это новый пункт): https://rizzoma.com/topic/a87a0a28b84d6326d4f3909e8801dab7/0_b_8nel_88gv1/

Надо её или использовать или удалить из репозитория. Я бы удалил

Определенно нужно разбить на два отдельных элемента.

Все подключаемые библиотеки импортить в тех компонентах, где они используются.
Jquery можно оставить глобально. И по возможности использовать import, а не require