Как и было сказано выше, в
<header> стоило запихнуть лишь логотип и навигацию, причем и само навигационное меню можно было бы обернуть в
<nav>, по семантике, так сказать. А все остальной контент сайта обернуть в тег
<main>, дабы уменьшить кол-во дивов на квадратный метр)
классы в li лучше назвать
class="menu-item", ибо в будущем верстка страниц явно будет более масштабной и название стилей по такому образу явно уменьшит шанс дублирования классов в разных местах страницы. Например, у тебя будет li меню и li товара буду иметь классы item, когда можно сделать как
class="navigation-item" и
class="product-item" соответственно.
По форме:
E-mail адресу можно было присвоить type="mail"
Не понимаю, почему у поля "Номер телефона" type="password", хотя есть для этого type="tel" (+ там ещё и опечатка в placeholder: Nuber- Number)
Чекбокс можно было сделать с помощью
<input type="checkbox" name="accept" id="accept">
<label for="accept">I accept the terms & conditions</label>
Обхват текста в тег
<label> и связыванию по id позволит нам активировать чекбокс, как кликнув по полю чекбокса, так и на сам текст
Так же, "terms & conditions" можно было бы обернуть в ссылку, которая, якобы при нажатии открывала эти условия соглашения в новом окне. Ну или если ты хочешь заморочиться, то можно сделал Pop up.
