Оцените первую вёрстку, пожалуйста?

Question

[Denis Limonov]

Добрый день. Сверстал первый макет и хотелось бы получить обратную связь, чтобы знать, что правильно, а что неправильно. Сайт вроде адаптивный, старался верстать по БЭМ. Слайдеры пока не делал. В общем, подскажите, что исправить, как улучшить и так далее.
Здесь ссылка на демо:
https://bronze-stone.github.io/
Здесь ссылка на файлы:
https://github.com/bronze-stone/bronze-stone.github.io
Спасибо :)

Comments

[WapSter]

В css никакого БЭМ'а нет стили элементов раскиданы.

[Denis Limonov]

Тимур Турсунбаев, ок, подучу БЭМ, спасибо

Answer 1

[adrenalinruslan]

Мой тебе небольшой совет, добавить к картинкам такой код CSS

border: 1px solid transparent;

А при наведении на картинку:

border: 1px solid цвет;

Что это дает ? Твои картинки не будут "прыгать", "двигаться" так сказать
У них по стандарту будет border с прозрачным цветом, а когда будешь наводиться, то цвет будет меняться, и эфект сдвига картинок не будет

Comments

[Denis Limonov]

Очень дельный совет, спасибо большое)

Answer 2

[Рональд Макдональд]

О, бизнес-линч, моя любимая тема.
Секси-бой на главной - на работе я б такое никогда не открыл. А дома открывал бы в приватной вкладке;

А нормально, что карусель не работают?

До цен и тренеров очень долго крутить колёсиком, ИМХО, не очень удобно.

Меню, которое открывается справа сверху, закрывает весь экран, чисто моё мнение - лучше, как на Тостере, чтобы только кусочек страницы.

Comments

[Denis Limonov]

На самом деле, это не мой дизайн, я только сверстал) Карусели не делал, потому что только недавно изучил html и css, к js еще не приступал. Но позже сделаю их. А вот меню пришлось придумать самому, так как в макете не было мобильной версии. Спасибо за замечание :)