[code-review] Верстка, какие замечания?

Question

[Паша Шворак]

Доброго времени суток.
Делал тестовое задания(линк) для одной фирмы и не получил никакого фидбэка. Я конечно понимаю, что очень большая конкуренция и может быть им не подошло мое резюме и тд. Но хотел бы, чтобы посмотрели на мою верстку. Есть ли какие замечания, которые сразу кидаются в глаза.
Буду благодарен за всю критику и советы.
upd.: ТЗ :
Был только psd макет и шрифты.

Comments

[Сергей Тутуков]

Тебя не взяли по двум причинам:
1) ux/ui дизайнер
2) если в ТЗ не написано использовать скрипты, это не означает, что они запрещены. Сделал бы имитацию ajax, валидацию формы, фильтр товаров, дропдауны нормальные тебя бы взяли

[Паша Шворак]

Сергей Тутуков: Спасибо, но хотел бы уточнить по первому пункту, что Вы имеете ввиду?

Answer 1

[Иван Вишневский]

• Жутко не хватает эффектов ховера на элементах.
  
• На фокусе нужно какое-то изменение поля e-mail.
  
• Слишком медленная анимация на ссылках — такие можно делать только там, где это по какой-то причине необходимо. Здесь же пользователь устанет быстро от такого.
  

Answer 2

[Анастасия Панфилова]

Я совсем не эксперт, но кое-что прямо очень явно бросается:
1. корзина. обычно при ховере что-то должно вылезать
2. поиск. ничего при клике не происходит, где строчка поиска
3. каталог. нужно было впихнуть плагин или самому написать переключение табов, чтобы заказчик видел как это будет реализовано в принципе
4. товары. обычно есть сразу кнопка добавления в корзину. не знаю, есть ли на макете, но я бы написала при ховере затемнение картинки и появляющуюся кнопку(пример)
5. лоад мор. опять же не работает, должна быть хоть какая-то демонстрация
6. отправка сообщения. я бы сделала аякс отправку и поставила модальное окно, стилизовав его максимально под макет
7. и конечно адаптив. нету адаптива, сейчас все подобные сайты адаптивны

По поводу самой вёрстки - я бы на твоём месте почитала про методологию БЭМ и начала использовать Bootstrap и не стеснялась бы использовать jquery/js
Это моё личное мнение

Comments

[Паша Шворак]

Спасибо, за ответ. К сожалению кроме самого макета никакого описания не было. Анимацию это я уже додумывал, чтобы хоть немного лучше смотрелось.

[Анастасия Панфилова]

Паша Шворак: к сожалению, так часто бывает. только один раз в жизни мне прислали макет, а там на слоях было описание как сделать анимацию, это было круто: )
заказчики часто не поощряют, а требуют "додумки":с

[Паша Шворак]

Анастасия Панфилова: Спасибо, учту)

Answer 3

[Smeecy Smeecy]

ТЗ бы скинул своё, а то мы откуда знаем по ТЗ ты сделал или нет :)

Comments

[Паша Шворак]

Прикрепил к вопросу.

Answer 4

[Сергей]

Не вижу заголовка h1.
В оформлении подписки, при клике по стрелке появляется синяя рамка вокруг ссылки, не думаю, что это эстетически верное решение.
Прописал бы title для ссылок, для сео.

Comments

[Паша Шворак]

Спасибо.