Правильно ли я верстаю?
Нет,не верно.
1.У тебя весь сайт лежит в теге <header>  .Логично же,что хедер должен быть только меню+лого в твоём варианте.В общем,тег хедера должен был закрыться перед этим- <div class="box">
2.Сайт не адаптивен,лучше используй фреймворк
3.Начни с структуры хтмл документа,так как у тебя изначально всё не так как должно быть.
4.Использование непонятно каких механик,у тебя банально сделаны отступы в меню прозрачным бордером: border-top: 8.5px solid transparent;
    border-bottom: 8.5px solid transparent;
    border-left: 20.505px solid transparent;
    border-right: 20.505px solid transparent;
Зачем это?
Ну и прочие ошибки...
Как и было сказано выше, в <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.