Поможете оценить верстку?

Question

[Александр]

Буду очень признателен, если вы краем глаза посмотрите на сайт. Верстал абсолютно без JS/PHP (еще основы учу - не хотел сразу использовать библиотеки). Верстал, к сожалению, для Yandex только(кроссбраузерность собирался с препроцессорами уже начинать осваивать), еще попробовал верстать по БЭМ(хотя модификаторы я не очень правильно использовал по итогу(не использовал)...). Также сделал +- адаптивный сайт, хотелось бы узнать ваше мнение и по этому поводу, да и в целом насчет всего ^_^
Собственно, макет(не pixelperfect, лишь для основной опоры), и codepen ('вредоносную' ссылку удалил - извините, моя оплошность D;)
Рад любой критике!!!

Comments

[Алексей Скляров]

Осторожно, поддельный сайт!
Посещение сайта s193070.smrtp.ru может привести к установке вредоносного ПО или хищению личной информации (например, паролей, телефонных номеров и данных банковских карт). Подробнее…

Чтобы улучшить режим Безопасного просмотра, вы можете отправлять системную информацию и содержимое страниц в Google. Политика конфиденциальности

[Александр]

Алексей Скляров, Да, видимо, временный хост - не лучшая идея. Удалю ссылку, оставлю codePen. Извините =)

[Рональд Макдональд]

Не, спс, бережёного Бог бережёт.

[Александр]

Извините, вредить людям не было мотива (хотел на временный хост посадить сайтик) - выставил на codePen на всякий =)

Answer 1

[Александр]

5/10
По пунктам:
1) Увеличение изображений не работает, как и слайдеры. Это работа верстальщика!
2)

Проблемы с адаптацией

3)

Не самый лучший результат в pageSpeed

4) Все стили со скриптом подгружаются в шапке сайта, что совершенно не правильно и может быть оправдано в редких случаях. Более правильным будет - вынести стили того что пользователь будет видеть в начале в тэг style перед html. В вашем случае вынести следовало все стили относящиеся к классу header и его деткам. Это, к слову, рекомендует гугл.
5) Изображения нужно ужимать, на кой черт мне грузить картинку в 4к если я сижу с телефона с 360р??? Это относится ко всем картинкам, ни одна не ужата.

П.С. Чего вы на парня взъелись за плашку с предупреждением? Да на все бесплатные хостинги такие вылезают, или ему для первых версток аренду брать?

Comments

[Александр]

Спасибо за разбор =)
1) Слайдер решил не делать, т.к. считал, что они на JS пишутся, которого я не знаю. С изображениями косяк - подтяну.
2) Скорее всего, вы имели навигационное меню - я его не стал делать в виде 'гамбургера' для мобильных версий, т.к. опять же не знаю JS =)
3) Как освою азы - обязательно начну работать над скоростью - пока что рано :)
4) Не знал о подобном решении со стилями - думал, только для JS подобное нужно.
5) Тут уже косяк серьезный =D Решил по быстрому первое изображение вставить.... С трезвой головой никогда такого не сделаю =)))

В общем, огромное спасибо. Без критики нет и роста ^_^

[Александр]

Александр, гамбургер можно сделать на чистом css, это ищется в гугле. Слайдер - это подключенная библиотека с одной строкой кода, которая есть на сайте этого слайдера, мало того,