Может кто-то проверить, что не так в моем тестовом задании?

Question

[errorpro]

Всем привет, для одной конторы делал тестовое задание, отправил, ответа так и не дождался. Т.к. у меня еще не много опыта, может кто-нибудь быстро глянет и скажет мне, какие у меня проблемы в знании верстки и что я не так сделал? Буду очень признателен.

Ссылка на псд и собственно тестовое задание.

https://yadi.sk/d/P_chBgEVcf7xy

Answer 1

[Сергей Протько]

Я бы расстроился как минимум изза rar. Залейте пример на plnkr.co

Из того что я вижу - у вас проблемы с размерами. Все намного меньше чем в макете. Более того - вот эта картинка в шапке судя по всему по задумке автора макета должна быть на всю ширину страницы а не фиксированным блоком. Как и хедер как и футер. Короче у вас сама структура макета не передана.

ID="features1" - славно, вы вкурсе что так делать не хорошо и вообще для стилей ID использовать не круто как минимум изза того что вы делаете неоправданно специфичный селектор а во вторых потому что у вас этот ID повторяется несколько раз на страницу нарушая тем самым его исконный смысл.

С активным статусом соц иконок тоже порадовали...

Словом выкинуть и переделать.

Comments

[errorpro]

Про ID точно, копипаста зло. Там надо класс и использовать. А что с активным статусом соц иконок?

[errorpro]

Они в псдшнике активные были, я оттуда и взял их.

[Сергей Протько]

errorpro: в макете вам показывают как должно быть при наведении курсором и тип курсора. А вы просто фигнули картинкой.

[Сергей Протько]

errorpro: просто не используйте ID. Вообще. Почитайте про специфичность селекторов, используйте препроцессоры стилей, почитайте про BEM и другие подходы в верстке...

То что я вижу сейчас можно сделать в фотошопе при помощи инструмента для нарезки.

[errorpro]

Сергей Протько: Аааа, блин, я думал они просто накосячили ;). Ну в целом на позицию джуниора я тяну? Или пока не дотягиваю?

[Сергей Протько]

errorpro: пока ранова-то. Практикуйтесь.

[errorpro]

Сергей Протько: Спасибо большое. Как думаете есть ли смысл переверстать и отправить заново?

[Сергей Протько]

errorpro: как минимум переделать все стоит. Возможно придется переделать все не один раз. Макет довольно простой, для практики хорошо. Даже если не отправлять его по месту требования (опять же можно с пометкой что б мол если не сложно что б разгромили) - то можно опять же тут поинтересоваться опять же.

[errorpro]

Ок, займусь. Огромное спасибо. И еще глупый вопрос, на планкер нельзя как-то импортировать свои файлы? Только открывать редактор и ручками все создавать?

Answer 2

[Тимофей Белоусов]

Во первых макет вообще не соответствует вашей работе, где обнуление margin padding??? Header и footer должен быть на всю ширину, после features нету очистки потока, в макете где нарисован курсор, дизайнер этим пытается показать вам, что будет происходить с элементом при наведение, блин если честно, тут можно будет целую статью написать по вашей работе, один совет, качайте макеты и практикуйтесь, набивайте руку...

Comments

[Сергей Протько]

судя по всему еще нужно человеку научиться понимать что в макете то...

[Тимофей Белоусов]

Сергей Протько: я с вами согласен, перед тем как начать верстать, надо уточни у дизайнера, что будет происходить при том или ином действие(клики или наведение), ну или самому не много подумать, к примеру те же соц кнопки в этом макете, видно же что все ссылки бежевого цвета(или другого не помню удалил уже макет!!!), а при наведение меняет цвет на зеленый. Набивайте руку и все!!! ПРАКТИКА, ПРАКТИКА и еще раз ПРАКТИКА!!!

[errorpro]

Про обнуление, поэтому у меня header к верху окна не прилипает? Читал в книге про обнуление, попробовал, но хидер почему-то так и остался.

[Тимофей Белоусов]

errorpro:
Как не работает??? Все работает!!! Вот это прописывай в самом начале стилей во всех твоих будущих проектах...

*{
margin: 0;
padding: 0;
}

[Тимофей Белоусов]

errorpro: И кстати ты молодец что сразу попытался в студию устроится, лишь на боевых проектах ты быстро начнешь развиваться, так как от тебя будут требовать, если не сдашь в сроки, уволят, мотивация)))) Но лучше пока набей руку на простых макетах!!! Прочитай книгу "большая книга css3" и вперед!

[errorpro]

Тимофей Белоусов: Хорошо, спасибо большое!