Раскритикуйте первую адаптивную (почти) верстку?

Question

[HealSpirit]

Всем привет! Наконец-то сделал свой первый psd макет, решил делать сразу адаптивный.
Брал макет отсюда, немного внес свои правки:
https://p.w3layouts.com/demos/aditii/web/
2 слайдера решил написать сам, не используя известные библиотеки, ну естественно они получились неадаптивными(
1 слайдер помогли сделать ( Слайдер с автоматической и ручной прокруткой? ), 2-ой удалось допилить самому , добавив автоматическую прокрутку и смена 1 слайда а не 3 ( https://learn.javascript.ru/task/carousel ).
Известные проблемы:
- неполная адаптивность (можно ли сделать самому такие слайдеры с адаптивностью?)
- отсутствие единой структуры наименования классов (начинал за здравие, закончил... как всегда)
- многое не дописывал (например, в @font-face, как я понял, нужно не только одно расширение шрифта, а несколько; гамбургер меню уже было лень доделовать и т.д)
- проблема с основными товарами (при перестройке с 4 в ряд до 3, хотя я думаю это нужно пропускать и выстраивать сразу в ряд по 2, потом по 1). Как я понял это решается адаптивностью, т.к. именно у основных товаров я не делал медиа запросы.
Вот верстка https://dimanik94.github.io/
Жду критики)

Comments

[Dima Polos]

Прикольно, молодец! Смотрел с мобилки, советую гамбургер отодвинуть от левого края и поиск сместить подальше, неудобно пальцем тыкать. Само меню побольше сделать, очень мелкие пункты, пальцем не попасть и шрифт очень мелкий.
Первое впечатление хорошее. Вопрос, свой адаптивный слайдер это принципиальная позиция или просто руку набить? Есть ли смысл? Для опыта - отлично, так и надо, а в бою, имхо, бери готовое.

Answer 1

[Shinso]

Тебе бы при адаптиве например

.slider .next {
    right: 0%;
}

.owl-carousel .next {
    right: 0%;
}

Для стрелок задать или вообще display: none если ненужны а то они добавляют прокрутку
Ну и для слайдера верхнего задать бы одинаковую высоту слайдов чтобы не дергало страницу при переключении слайдов.