AndrewMarty
@AndrewMarty
Верстальщик

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

Здравствуйте, прошу Вас оценить мою верстку - https://andrewmarty.github.io/Travel/.
Может у Вас есть какие-то рекомендации по наименованию классов или адаптивности.
  • Вопрос задан
  • 160 просмотров
Пригласить эксперта
Ответы на вопрос 4
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
Начинающий верстальщик
Нормально, анимация тоже нормальная, нет размытости как по мне, где брал макет если не секрет?
Ответ написан
sashabeep
@sashabeep
Дней без вопросов про Slick на тостере : 0
Стандартное г...о без задач.
Разве что везде в текстовых блоках у параграфов добавлены классы, и если убрать класс - параграф скрывается вовсе, как будто его нет. Так вот, ни один из технологов не будет заниматься тем, чтобы в визуальный редактор в админке(а вы ведь догадываетесь, что этот шаблон будет подключаться к какой-то админке) подкидывались ваши классы к параграфам.

Если пункт меню ведет на текущую страницу - будьте добры указать ссылку по-нормальному, с #. Сейчас у вас там попытка подменить ссылку скриптом. Если ее скопировать или открыть в новой вкладке - 404

Зачем лишний тег-обертка main?

Неизвестное значение у атрибута lang в html
Ответ написан
Ваш ответ на вопрос

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

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