visill / group_project

Групповой проект интенсива по Django
1 stars 0 forks source link

МНОГО замечаний по шаблонам #25

Closed drowsycoder closed 1 year ago

drowsycoder commented 1 year ago

Остался пустой homepage.html Футер footer.html, кстати, тоже пустой

Система отступов в шаблонах тотально нарушена Для начала просто соблюдайте отступы в 2 пробела, а не прыгайте с 2 на 4 и наоборот Потом учитывайте структурные джанги-элементы как часть разметки

Инклюды стоит располагать в своих приложениях, к которым они логически относятся. Я про event_card.html и exhibit_card.html А вот зачем в отдельный иклюд вынесена строчка с загрузкой иконки сайта, которая должна бы отображаться в заголовке вкладки, - это спорный вопрос Как и homepage_greetings.html, который можно было не выносить отдельно (там же только абзац-приветствие) Вот если бы вынесли карточку музея, несмотря на непереиспользуемость, - здесь была бы логика

В base.html: {% block content %}Some content{% endblock %}. Зачем такой плейсхолдер? Просто грамотно используйте шаблон и в дочерних подставляйте нужный контент В ту же степь - бесполезный mainpage.html. Стоит лучше подумать над структурой

В header.html: href="{% url 'museums:museum_list'%}": содержимое джанго-тегов отбивается одинарным пробелом (перед закрывающей конструкцией пропущен)

hecker0makhambet commented 1 year ago

Сделано