@shyzooo
Начинающий frontend

Оцените Верстку?

С мая изучаю html/css,сверстал 2 макет,прошу оценить и подсказать,где у меня основные ошибки,что подучить и так далее.Заранее Спасибо!! https://yadi.sk/d/KjtiZzgiekLYYg
  • Вопрос задан
  • 3375 просмотров
Пригласить эксперта
Ответы на вопрос 4
@amangida
Хорошо, но есть недочёт в виде картинок, которые вылазят за рамки контейнера. Вы указали max-width для контейнера, равный 78%, но не указали max-width: 100% для картинки в нём.5d3416949d760482213738.png
Ответ написан
Комментировать
@KrajeBieter
Стрелка какая-то странная, при наведении подсвечивается только часть, и вообще, на неё сложно навестись:
5d354590a38e3819988334.jpeg
И здесь не вся подсвечивается, а только круг
5d35482628f20100352627.jpeg
Коэффициент контраста надписей и фона на некоторых элементах слишком низок, с точки зрения доступности (accessibility) это плохо.
5d354784a3bf8257454339.jpeg
5d35478b4df4c769062733.jpeg
5d3547923cefc329644875.jpeg
5d3547c30b119298149023.jpeg
И что такие шакальные картинки, на них прямо артефакты сжатия видны, используй другие методы сжатия например

А ещё
почему-то адблоком блокируются эти пикчи5d354a8d8d26c470956426.jpeg
Ответ написан
@Vanitson
Frontend developer
В целом неплохо. Когда я учился, у меня второй макет был намного хуже)
5d35f0a29aa43283821150.png
Как-то совсем топорно сделаны стрелки. Если не критично, можете использовать Вот это для выбора стрелок на слайдер например. Не используйте цифры в названиях классов. Альтернатива: "arrow-left-top", "arrow-right-top".
Почему не использовали bootstrap (лично мое мнение, с ним было бы проще, много одинаковых элементов расположенных горизонтально)? Принципиально, или еще не знакомы?
5d35f0d587fb1510329666.png
Почему padding а не align-items: center? Используйте флекс по полной, это мощный инструмент который упрощает жизнь.
Свойство transition устанавливайте на сам элемент, а не на hover, тогда анимация будет плавная и при наведении, и при уводе курсора с элемента.
Далее все более менее, как для второго макета, очень даже неплохо.
Повторюсь, хорошо ознакомьтесь с флексами, есть много мест в вашем коде, которые можно было сделать проще с помощью этого инструмента.
P.S. - Не стоит воспринимать мои слова как истину, уверен есть много людей которые сделали бы лучше, чем так как я написал, я просто выражаю мнение на основе своего опыта.
Ответ написан
webdesignermsk
@webdesignermsk
Веб-дизайнер, программист, креативщик, сценарист
Отсутствует всё, не нужно шаблонить.
Ответ написан
Ваш ответ на вопрос

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

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