Что скажете по поводу верстки?

Question

[Nevada18]

Здравствуйте. Сверстал такой вотсайт.
Кому не сложно - оцените и озвучьте допущенные мною ошибки (про alt знаю, просто забыл его заполнить). Так же приветствуется критика. И еще вопрос - стоило ли делать фильтр и оценку блоков (всплывающие звездочки при наведении)? Имеется ввиду перестроение блоков по нажатию на фильтр.

Благодарю за внимание, оценку и критику!

Answer 1

[Сергей delphinpro]

1. h1, h2, h3, h4, h5, h6, p, a { display: inline-block; } - весьма неожиданно. Зачем так глобально переопределять?
2. Обнуление списков конечно удобно, но как быть с отображением списков? (обнулять нужно конкретные классы)

ну а так в целом норм.

Comments

[Nevada18]

1. чем это плохо?
2. Спасибо, учту. Забыл про смену контента)

Answer 2

[Nwton]

Код не читаемый. Сравни:
1) Твой
2) Как должно быть

Если представить, что макет не твой и сайтом никто не будет пользоваться, то сделано не плохо. А иначе очень много совершенно очевидных проблем:
1) При наведении на звездочку, сайт должен говорить мне, что я на нее навел. Как минимум заливкой
2) Почему при наведении на кнопку Login, она подсвечивается, а аналогичная кнопка поиска - нет?
3) Так не должно быть. Блок должен занимать всю высоту
4) Для чего вообще outline?
5) Слишком много элементов с маленьким шрифтом, которые к тому же сливаются с фоном
6) Простой serif не подходит для кнопок, используй sans serif

И так далее. Но для начала довольно неплохо.

Comments

[Сергей delphinpro]

Это код после препроцессора скорее всего =)
Вряд ли чел сидел, заморачивался с форматированием.

[Nevada18]

1 и 3 - спасибо, учту.
2. упустил этот момент)
4. это элемент при фокусе, чтобы показать местонахождение при использовании кнопки tab. как тогда сделать это лучше?
5. таков макет) на каком разрешении вы смотрели?
6. понял, благодарю.

[Nwton]

Nevada18: 1280x800 ноут