Оцените верстку шаблона?

Question

[Valeri-afk]

Начал заниматься версткой месяца 2 назад, поэтому не совсем разбираюсь, насколько правильно сделана верстка и написан код, так что нужно, чтобы разбирающиеся люди помогли советом.
https://valeri-afk.github.io/Valeri-afk.github.io3/

Comments

[Froggyweb]

Ну двоякое впечатление
в целом контент - норм. на мобилках не хватает классов бутстрап например в разделе Discover Our Services
шапка сайта на мобилках тоже разваливается
в целом - приятно

[FinGanapre]

Шапка на мобилке выглядит сомнительно (Сайт обычно должен хорошо выглядеть на экранах от 320px и выше)
Телефонный номер должен быть кликабельным для звонка. href="tel:[phone]", почта href="mailto:[e-mail]".
Иконка поиска в шапке должна быть обёнута в кнопку, если она открывает блок с поиском. Если это ссылка на страницу поиска, то это должна быть ссылка.
READ MORE в блоке About US cursor: pointer. Картинкам на мобилке не хватает отступа между, но это не точно (не видел макет).
Discover Our Services: на мобильной версии 3 колонки это много, лучше свернуть в одну.
What Customers Say?: на мобилке не помещается
Our Blog & Event: скорее всего, весь блок карточки это сслыка. Т.е. вы просто обёртываете его в тег а. Сейчас это div.
Телефон и почта в футере это тоже ссылки. Адрес скорее всего ссылка на страницу с контактами и картой.

Очень много пикселей. Почитайте про rem и em в дальнейшем. С определённого разрешения есть горизонтальный скрол, найдите элмент, который не вмещается. Можно удалять по очереди блоки, чтобы быстрее найти.
Хорошая работа для двух месяцев:)

[generate]

Ничё без бутстрапа не могут

Answer 1

[Lukin Danil]

Если тебя интересует правильность написания кода, то можно проверить тут . А если со стороны пользователя, то анимация появления мобильного довольно долгая как по мне, я обычно ставлю 0.2s, а вот анимации на исчезновение вообще нету. Еще в дектопной версии меню по вертикали не совсем по центру, для этого советую ознакомится с CSS-свойством transform. ну или padding(ами) просто вверх и вниз сделать и она сама по вертикали встанет. Для поднятия удобства пользования меню фиксированным можно сделать. Значки соц. сетей в шапке чуть чуть побольше бы сделать, ну или сами ссылки увеличить, например мне не совсем комфортно наводить на них. А так для 2ух месяцев очень даже не плохо, молодец :D

Answer 2

[Герман Зуяков]

Анимации добавь, слайдер, калндарь оформи, убери оутлайн у всех елементов, слайдер не работает
и адаптив подхрамывает

Answer 3

[Sergey Brin]

где учился?

Answer 4

[Vyach Gor]

Где хоть один ховер? В навигации ховер должен соответствовать активному классу (желтая полоска внизу), а не просто text-decoration: underline; Есть пару "мертвых" элементов по типу иконки поиска и селекта выбора языка - обычно такие элементы также нужно делать, даже если это никак не предусмотрено в дизайне.

Answer 5

[Богдан Лесько]

В плане удобства на смартфонах в шапке нужно увеличить отступы между элементами и неплохо бы увеличить немного размеры иконок.

На всей странице убрать горизонтальный скролл