Оцените вёрстку?

Здравствуйте, прошу Вас оценить мою верстку - https://andrewmarty.github.io/Travel/.
Может у Вас есть какие-то рекомендации по наименованию классов или адаптивности.
  • Вопрос задан
  • 204 просмотра
Пригласить эксперта
Ответы на вопрос 3
fallus
@fallus
1. Я бы ещё ко всем <img />применил css-свойство image-rendering, подбирая для каждого браузера своё значение префиксами.

2. В .slider__ я бы / поместил в отдельный <span />
.slider__current и .slider__max задал бы минимальную ширину (дав им inline-block, а лучше их контейнеру flex).
Чтобы не дёргалось вот это вот всё тогда, когда числа меняются при смене слайда.

3. Было бы лучше, если б это была также и респонсив вёрстка(в rem). Потому что если уменьшить масштаб страницы, сайт уже выглядит не так, как на десктопе. Уже. Меньше. Хотя не знаю, может так было задумано.
Да и подобного сайта, наверное, достаточно и в пискселях делать.

4. На мобильной версии, вместо того чтобы давать .header position: fixed, я бы изначально дал position: sticky. А уже классом .header_fixed просто давал бы background и box-shadow.

5. Ну а подобные анимации я никогда не понимал. Не нравятся субъективно.
Ответ написан
lamer350
@lamer350
กำลังสูงสุด
Норм, ужасные подходы только к анимации, плывет в глазах от того что все это разъезжается с разных сторон, это все портит. Но чисто как для практики ок.
Ответ написан
Комментировать
@Vladneim
Начинающий верстальщик
Нормально, анимация тоже нормальная, нет размытости как по мне, где брал макет если не секрет?
Ответ написан
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Похожие вопросы