Оцените первую верстку

[Homo Habilis]

http://hhabilis.github.io/greenish/index.html

[parnas]

Вроде неплохо. Но есть замечании.

1.Шрифты подключите до стилей. К тому же нужно сперва подключить ресеты(normalize.css). У вас наоборот.

2.Придирка конечно но будеть полезно: научитесь делать меню пункты которого независимо от количество пунктей должны тянуться на всю ширину. Тоесть если сейчас удалить один из пунктов меню у вас образуеться пустое место. этого не должно быть. Ну конечно,НЕ всегда это имеет значение.

3.К пунктам latest projects если добавить снова 6 пунктов, у вас они не поместяться в 2 ряда. Так как у вас они inline-block, научитесь справляться с пробелами.

Edited July 17, 2014 by parnas

[vladmih]

1. Кнопка является дивом, а внутри ссылка, соответственно, юзеру нужно тыкать именно на текст, что есть не юзер-френд, и ховеры!!! Ребят, ховеры! Не видно обратной связи с пользователем.
2. в лого div не обязателен, можно было бы просто к ссылке этот класс применить...
3. Как придирка, не понятно как выглядит активный пункт меню... Подразумеваю что так же...
4. banner я бы сделал так... Див с классом баннер, внутри картинка, и там же лежит заголовок и span к примеру, в итоге убираем классы лишний див обвертки и пишем стили так banner > h1, banner > span, banner > img
5 .feature_row , galery_row , info_row, client_row  я бы сделал списками... Я думаю это более было бы семантичнее ИМХО.
6 footer_menu булиты квадратные есть и в css, не вижу смысла делать картинкой... На крайняк, псевдоклассами... Убираем лишний запрос к серверу на картинку...

Возможно придирки, но я бы сделал так...

Для первой в целом, не плохо! Моя первая была вообще писец...
 

Edited July 21, 2014 by vladmih