Скажите что с версткой не так, где ошибки?

Question

[Денис Липатов]

Хочу критики, направьте на ошибки, а то я чего-то не догоняю, что не так то? вот верстка тестового, скажите что в коде не так?
https://yadi.sk/d/SHGaGXIlvjfmr

Comments

[Ankhena]

Выложите куда-нибудь не архивом, на хостинг или https://jsfiddle.net/

И почему Вы решили, что в ней что-то не так?

[Денис Липатов]

Ankhena W:
demo.pp.ua/test

Это тестовое задание, нужно было просто сверстать, не натягивать на WP, ничего лишнего...я выполнил задание, и мне прислали: "К сожалению, по итогам его оценки мы не можем предложить Вам работу. На данный момент ваш ответ не соответствует требованиям компании."
Я конечно не супер профи, но хорошо сверстать и адаптировать могу без лишнего мусора в коде...

[xtala zen]

Денис Липатов: это может быть обычная отписка на все случаи жизни и дело вовсе не в верстке. Или еще как вариант вы сделали кому-то бесплатно работу)))

[Сергей]

Как сказали выше - вы за кого-то поработали и это 100% верное утверждение. У вас же есть портфолио? Показывайте его и просите оценивать по нему.

Answer 1

[Zam_Bo]

Ну, я, например, не люблю большой каскад, который местами у Вас встречается.

Comments

[Денис Липатов]

Где например, если не сложно, поделитесь опытом)

[Zam_Bo]

Например такой селектор .slid-baner .sckreen .plushka .code span
+ сам класс .plushka - это же транслитерация? Лучше все-таки английским из называть.
Еще заметил .slid-baner .snippet .hotel-info .block-r и его стили
{
display: inline-block;
float: left;
}
Я бы либо с inline-block либо с float сделал, но использовать оба вместе неуместно)
В общем так)

Answer 2

[Raz-Dva]

Я конечно не специалист(работаю верстальщиком где-то мес. 5 ) но скажу о тех ошибках на которые сам попадался.
-если добавиться номер тел. Лучше сделать div.tel>p(или span)и потом проверить добавляя или удаляя номера телефонов
-в шапке я сделал бы не через col а просто дивы в одну строку и выровнял бы через vertical-align:midlle. В таком случае не нужны марджины вверху и блоки всегда были бы вертикально выровнены посередине.
- спрайты делаю в одном файле. Не надо разбивать для соц сетей, для звездочек. А постараться в одном файле их сделать.
Ну и нужно всегда проверять на количество текста(если много текста или мало), так же с изображениями(если вставят с админки большое или меньшее изображение) не полезет ли верстка. Цена тоже может меняться. Это как моё мнение. Я думаю что более опытные найдут еще что-то, просто не всем хочеться тратить свое время.