Правильно ли использован Bootstrap?

Question

[Sanan Yuzb]

Здравствуйте уважаемые форумчане!!!

Неделю назад решил попробовать Bootsrap. Поэтому решил сверстать свой первый макет используя этот фреймворк.
Ссылка на сайт : samplesite.esy.es Прошу критики на счет использования Bootsrap и верстки в целом. Слайдеры не делал, т.к. по не хорошо усвоил javascript.

Заранее благодарен.

Answer 1

[Aleksandr]

Зачем фоновые картинки делать png??
Они ж весят хз сколько.

и меню выглядит ужасно (когда бургер)

Comments

[Sanan Yuzb]

А в каком формате хранить?

[Sanan Yuzb]

Меню, да согласен, надо будет исправить.

[Aleksandr]

Sanan Yuzb: jpg (кач-во 50-60)

[Sanan Yuzb]

Александр Котов: Ок спасибо, учту.

[Aleksandr]

Sanan Yuzb: также мне кажется, что вы неправильно используете h1 ))

[Aleksandr]

Sanan Yuzb: и интересно, почему вы решили использовать vw?

[Sanan Yuzb]

в каком смысле неправильно? vw для адаптивности шрифта

[Aleksandr]

Sanan Yuzb: ну в блоке Latest Products не нужно использовать h1

[Sanan Yuzb]

Александр Котов: Понятно

[Aleksandr]

Sanan Yuzb: ну, а Bootstrap правильно использован

[Sanan Yuzb]

Александр Котов: Спасибо. А саму верстку в целом по 10 бальной шкале на сколько оцените?

[Aleksandr]

Sanan Yuzb: ну если убрать все минусы, то очень хорошо. 10

[Sanan Yuzb]

Александр Котов: Спасибо. И последний вопрос, стоит ли использовать vw для шрифтов или есть более оптимальный метод?

[Aleksandr]

Sanan Yuzb: em?

[Sanan Yuzb]

Александр Котов: Да точно, спасибо!!!

[Aleksandr]

Sanan Yuzb: если вы получили ответ, закрывайте вопрос.

Answer 2

[Kostya Gorozhanov]

Восемь заголовков h1
большинство картинок надо было на фон ставить а не играться с position.
"40% Off New Autumn Collection" - этот параграф вообще изнасилован.
одним словом вам надо отучиться строить все с помощью position, и начать использовать сетку.

Comments

[Sanan Yuzb]

А как делать адаптивные изображения, если ставить на фон блока (напрмер 3 блока col-md-4)? Возникают проблемы с высотой.

[Kostya Gorozhanov]

Sanan Yuzb: я обычно использую background-size:cover

Answer 3

[Андрей]

Еще конечно такие вещи, как слайдер, сразу делать, а не понатыкать точек как будто это он, лишняя работа же)

Answer 4

[Даниил Сорокин]

И в чем прикол тут использования бутстрапа? Вы тупо взяли и написали всем класс xs, и у вас получилась просто резиновая верстка которая на мобильных устройствах хренова смотриться. Нужно чтобы блоки watches,men,women отражались на всю ширину экрана на мобильных устройствах, ибо это уже не адаптивная верстка.

Comments

[Sanan Yuzb]

Согласен, а как быть с изображениями они огромные при маленьких размерах экрана? Придется их хранить в нескольких размерах и подключать для каждого размера экрана?

[Даниил Сорокин]

Ну они и должны занимать все ширину экрану посути. Пропишите при маленьких экранах не max-width 100% а width: 100%