Структура страниц

Everrest1 commented 4 years ago

Pug позволяет использовать отдельную страницу template, например, для шапки, хэдера и футера, а затем вызывать этот темплейт на разных страницах, размещая на них только блок с контентом

Everrest1 commented 4 years ago

Это должен делать сборщик автоматически, нет?

Everrest1 commented 4 years ago

Это должно быть папке с блоком

n0sfer666 commented 4 years ago

Это должен делать сборщик автоматически, нет?

Переместил script(src="main.min.js") в компонент head (изначально, так и было, но в прошлых ишью переделывал не помню уже зачем) (d0e85d5) Переделал на автоматическую инъекцию (c68f731)

n0sfer666 commented 4 years ago

Это должно быть папке с блоком

Перенёс в date-dropdown-init.js (7391676)

n0sfer666 commented 4 years ago

Pug позволяет использовать отдельную страницу template, например, для шапки, хэдера и футера, а затем вызывать этот темплейт на разных страницах, размещая на них только блок с контентом

Создал template (components/template/template.pug) (e589d19). Отрефакторил страницы (66d978b).

Everrest1 commented 4 years ago

Тег body мы не трогаем

Здесь теги header и footer кажутся лишними

Everrest1 commented 4 years ago

template.pug стоит перенести к остальным страницам

Everrest1 commented 4 years ago

Думаю, лучше будет сразу передавать литерал объекта в миксин. В читаемости не сильно большая разница, но хотя бы не нужно объявлять лишних переменных

Everrest1 commented 4 years ago

Старый код)

Everrest1 commented 4 years ago

Тут тоже не понятно зачем такая структура

n0sfer666 commented 4 years ago

Всё исправил. Сделал 2 отдельных шаблона template-ui-kit и template-page (удобнее получается)

Everrest1 commented 4 years ago

Может лучше делать импорт, там где эта компонента непосредственно используется?

Everrest1 commented 4 years ago

Просто класс из неоткуда

Everrest1 commented 4 years ago

Это теперь лишний блок?

n0sfer666 commented 4 years ago

Может лучше делать импорт, там где эта компонента непосредственно используется?

Сделал (711d6e8, c1bc51f). Так лучше (быстрее компиляция)

Просто класс из неоткуда

Удалил (14cb0dc)

Это теперь лишний блок?

Удалил (070311e)

Everrest1 commented 4 years ago

Откуда здесь взялся .inline-datepicker.js-inline-datepicker? Модификатор - это имя прилагательное в английском языке. По стандартам п.1 https://github.com/fullstack-development/front-end-best-practices/blob/master/CSS/bem.md

Everrest1 commented 4 years ago

Имя класса и файла не совпадают

Everrest1 commented 4 years ago

Почему бы не дать тегу a этот класс?

n0sfer666 commented 4 years ago

Почему бы не дать тегу a этот класс?

Отрефакторил сначала неправильно (d24a834), потом правильно, надеюсь (83ace31)

Имя класса и файла не совпадают

Переименовал (8cd284c)

n0sfer666 commented 4 years ago

Откуда здесь взялся .inline-datepicker.js-inline-datepicker? Модификатор - это имя прилагательное в английском языке. По стандартам п.1 https://github.com/fullstack-development/front-end-best-practices/blob/master/CSS/bem.md

Сделал .date-dropdown_inline (2eccb28). Кстати, не знаешь как туда кнопки засунуть? (onShow не работает с inline: true)

Everrest1 commented 4 years ago

Не совпадают имена

Everrest1 commented 4 years ago

Модификатор _320-px не соответствует именованию БЭМ Модификатор date-dropdown_inline используется вне своего блока

n0sfer666 commented 4 years ago

Не совпадают имена

Переименовал файлы (2bb1191)

n0sfer666 commented 4 years ago

Модификатор _320-px не соответствует именованию БЭМ Модификатор date-dropdown_inline используется вне своего блока

Поправил (a9ed6d8, 246892c)

Everrest1 commented 4 years ago

js-pagination вне своего блока или без миксина

n0sfer666 commented 4 years ago

js-pagination вне своего блока или без миксина

Создал миксин и отрефакторил страницы (5bd4b7d) UPD. Немного переделал (fd1d7e7)

n0sfer666 / Toxin-hotel

Структура страниц #38