Оцените часть верстки 1-ого макета?

Question

[gleendo]

В общем решил я все таки сверстать макет. Оцените пожалуйста небольшую часть верстки.
Ссылка на скачивание верстки: https://cloud.mail.ru/public/6zpY/2XveKTQnF
Дайте адекватную критику, чтобы в дальнейшем, верстка была лучше.
Заранее спасибо!

ps. Времени на эту часть ушло примерно 2 часа. Нормально ли для первого раза?

Comments

[Андрей Федоров]

Где адаптивность? Учтете, это все равно потребуется, а значит вам придется переделывать шаблон.

[gleendo]

Андрей Федоров: Мне сейчас не до адаптивности. Это ведь первый макет который я решился сверстать.

[Андрей Федоров]

con_: Ну, а что тогда оценивать? Выглядит аккуратно для фиксированной верстки. Правда в html нет семантики - header, sidebar и т.п...

[Serj-One]

Хочешь советов, выкладывай вёрстку на хостинг. Качать твой архив никому не нужно.

[Anton Filippov]

что еще предложишь скачать? хости на гитхабе

Answer 1

[Денис Атясов]

Если смотреть код css то зачем и html и body присваивать фон одновременно? Во вторых, у вас background: cover не работает, потому что его переопределяет следущая строка:

html, body {
	background: cover;
	background: url(../images/background.jpg) no-repeat center fixed;
}

Ну и конечно wrapper с шириной 1512px и фиксированной высотой 1016px явно что-то пошло не так.

Далее весь код из звездочки кроме box-sizing, рекомендую перенести в body:

* {
	box-sizing: border-box;
	color: #485254;
	font-family: 'Roboto';
	font-size: 16px;
	font-weight: 400;
	line-height: 24px;
}

Comments

[gleendo]

background-cover работало, до того как все свойства по алфавиту отсортировал. А как еще задавать размеры wrapper?

[Денис Атясов]

background: url(../images/background.jpg) no-repeat center fixed;
background-size: cover;

Answer 2

[Dave]

Для первого раза, тем более за 2 часа - молодец. Твердая 5-ка. А фон фикс или не фикс - это формальности. CSS и HTML оформленны очень хорошо.