@Dmz_wow

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

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

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

Войти через центр авторизации
Похожие вопросы
Charltons Гонконг
от 150 000 до 180 000 руб.
Россайт Санкт-Петербург
от 60 000 до 60 000 руб.
24 июн. 2019, в 15:25
500 руб./за проект
24 июн. 2019, в 12:59
15000 руб./за проект