Оцените!
Rieltor
Rieltor исход
- .logo наползает на блок .main. Не ограничивайте высоту header. Снова не хватает распорки.
- Для активного пункта меню используется фоновое изображение. Гораздо проще было залить сплошным фоном или градиентом, а не использовать дополнительную картинку.
- Лучше, если один из пунктов меню будет выбран по умолчанию при загрузке страницы. А то пустое место смотрится не очень.
- В блоке .hidden есть перечень “преимуществ” что-ли. Почему в разметке это оформлено не как список ul?
- Немного странно, что подвал состоит только из формы отправки заявки, но что поделать, видимо такой макет.
- Изображение девушки в данном случае скорее
<img>, чем фон. - Зачем текст подписи оборачивается дополнительно в параграф? И если label связано с полем по id, то поле в label оборачивать уже не надо.
- Фоновая картинка для кнопки отправки формы… Почему не градиент?
- Для чего кроме валидации данных из полей формы используется подключенная библиотека jQuery?
1,Не понял куда вписать clearfix
4.А мусора не будет?
7.Тыкаешь на <p> текст и input уже ждёт чтоб ты написал.
9.Чтоб ссылка не была активна.(Тыкаешь на <a> а он тащит страницу вверх)
1.Для header.
4.Какого мусора? Чем вас не устроил список?
7.Тыкаешь на label (в этом и есть смысл связи по id). Тег <p> лишний.
9. Поняла. Можно было использовать более простую проверку, без js. С помощью атрибута required отметить поля обязательные для заполнения.
Что касается ссылки, то можно отменить действие по умолчанию с помощью e.preventDefault(); И библиотека будет не нужна.
Гляньте css строку 92 он не работает.