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

Здравствуйте, прошу Вас оценить мою верстку - https://andrewmarty.github.io/Travel/.
Может у Вас есть какие-то рекомендации по наименованию классов или адаптивности.
  • Вопрос задан
  • 202 просмотра
Пригласить эксперта
Ответы на вопрос 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
Начинающий верстальщик
Нормально, анимация тоже нормальная, нет размытости как по мне, где брал макет если не секрет?
Ответ написан
Ваш ответ на вопрос

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

Войти через центр авторизации
Похожие вопросы