@Dmz_wow

Правильно ли я верстаю?

сверстал шапку макета и понял,что кажется я верстаю не так как надо.
1.хотелось бы получить советов
2.указаний на ошибки
3.и сказать чего не хватает(что подучить)
вот ссылка rollerstore.ru/?login=&pass=
  • Вопрос задан
  • 506 просмотров
Пригласить эксперта
Ответы на вопрос 3
@coverme
Нет,не верно.
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;

Зачем это?
Ну и прочие ошибки...
Ответ написан
@cheese13
Подучи flexbox.
Тебе станет значительно проще позиционировать элементы и проще осваивать адаптивную верстку.
Ответ написан
Fdech
@Fdech
Верстальщик
Как и было сказано выше, в <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.
Ответ написан
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через TM ID
Похожие вопросы