@AnaMontanher
Denis Matern
@d-maternAll comments
- @d-matern
- неверная семантика, тег h2 применяется для локации, хотя его лучше применить для описания профессии;
- вместо div для контейнера, я бы применил main;
- <div class="attribution"> этот блок нам не нужен при выполнении;
- fonts подключаются из папки assets/fonts.
- @medievalnudistofthedead@d-matern
- стили должны быть подключены из папки assets/fonts;
- стоит изменить семантику html;
- нам не нужен footer;
- верстка без адаптивного дизайна;
- стили можно упростить.
- @kaffeenj@d-matern
- стоит подумать над применением тега "main" для лучшей семантики;
- использовать CSS Flex or CSS Grid для позиционирования контента вместо margin;
- высоту контейнера задать с помощью "vh";
- для заголовка использовать "h1" вместо тега "p"
- макет выглядит не соответствующе дизайну;
- css можно упростить.
