Оцените пожалуйста верстку, что я делаю не так?

Question

[VegasChickiChicki]

Я предполагаю что таких тем тут уже много ,но все же и я решил создать такую. Все как обычно , оцените ,помогите мне найти ошибки в том что я делаю ,в каких местах костыли ,в каких местах лучше переделать и так далее. Еще хотелось бы ,что бы объяснили о преимуществах SASS ,а то я вроде бы использую его ,а все плюсы ,которые я нашел ,это не нужно ставить скобки и точку с запятой :) ,а как более эффективно его использовать в верстке ,я пока не могу понять(
По поводу JS все просто , все писал сам ,но поскольку в нем мне пока сложно ориентироваться , вышел он не совсем грамотным ,хотя и работает.
Ссылка на гит https://github.com/VegasChickiChicki/VegasChickiCh...
Ссылка на сайт https://vegaschickichicki.github.io

Answer 1

[Анатолий]

Для начала сайт очень тяжелый 15Мб, надо снизить хотя бы максимум до .5Мб.

PS. По критике, посмотри все ролики https://www.youtube.com/channel/UCaTfYudJUVA8cV_Bu...
потом сам оценишь свой сайт.

Answer 2

[lamer350]

К сожалению, вам еще рано выносить свои работы для оценки.

... что я делаю не так?

В ответ на ваш вопрос - можно сказать одно: Вы не учитесь, а хотите чтобы мы все сделали за вас.
Начать стоит с того что материалов по SASS в интернете больше чем нужно, вы видимо даже не пытались разобраться с этим.
По сайту можно сделать вывод что вы так не разбирались даже с HTML, вы даже не умеете использовать теги HTML (речь именно о HTML5).
Отдельный вопрос стоит уделить графике сайта, после того как я открыл фон в ~7000 пикселей в высоту дальше не стал смотреть, особенно выбор PNG смутил.
Разбирать в вашей работе просто нечего, начать нужно с основ. О JS вообще даже упоминать не хочется.

Comments

[VegasChickiChicki]

По поводу SASS понял ,буду читать ,учить. По поводу HTML не понял ,в чем конкретно мои ошибки ,можете указать на них? Я ведь писал этот вопрос ,с целью поиска ошибок ,а вы сказали что-то вроде "Все плохо ,иди учись дальше ,ошибок море ,каких ,естественно ,я не скажу". По поводу фона , в макете был общий бекграунд картинкой в ~7000 пикселей , что делать с таким макетом? Мне ее резать или как то менять? Я не могу понять.
И по поводу PNG, я так понял нужно для фото использовать другой формат? Я пытался сжать фото ,выходит им еще лучше формат поменять ? Если да ,то тогда на какой?

[lamer350]

VegasChickiChicki,

По поводу HTML не понял ,в чем конкретно мои ошибки

В этом и суть моего ответа, вы не учитесь и не пытаетесь учится. Ждете когда вас носом будут тыкать?
У вас там всего 2 тега HTML5, прошел час с моего ответа и вы даже не погуглили чтобы изучить в каких местах используются эти теги.
Надеюсь вы понимаете к чему я веду? Поймите что вот это все нужно только вам, ни мне, ни кому другому. Если вы так относитесь к своему образованию, я за вас гуглить материал и тыкать носом не буду. Описать в 2х словах это все не получится. Сейчас я вам скажу что нужно один формат использовать для графики, но это будет конкретно один случай, не значит что в остальных случаях нужно будет так же поступать... Потому нужно понимать основы, даже с графикой. Материалов вагон и маленькая тележка. Но вам видимо лень.
Понимаете? Нечего разбирать, одно дело когда есть хорошая работа, человек старался изо всех сил - тогда можно что то разбирать, советовать, спорить. А другое дело когда человеку лень. Открывая вашу работу сразу видно халтуру. Потому единственный вариант помочь вам - это посоветовать открыть теорию, не поленится. Даже если лень читать, смотрите ролики на ютубе, а начать стоит с нуля!

[VegasChickiChicki]

lamer350, Отнюдь ,я погуглил и посмотрел и нашел в интернете вагон и маленькую тележку тегов HTML 5 ,я не могу понять ,куда вы мне предлагаете их растыкать? Возможно стоит убрать тег header , возможно стоит основные блоки поместить в section. По поводу изображений понял , стоит их оптимизировать и выбрать правильные форматы.

[Дархан]

VegasChickiChicki, кажется он говорит о семантической верстке,но насколько это критично вопрос другой.
Картинки сжимали? compress png тот же.

[VegasChickiChicki]

Дархан Камалиев, Сжимал ,но на сайте их просто очень много...Буду сейчас читать по поводу форматов ,возможно стоит поменять формат фотографий.

[Даниил Маслов]

VegasChickiChicki, заголовок h1 сделан по приоритету размера шрифта, а не смысловой нагрузки.
alt к картинкам прописан скудно.
тянете целый fontawesome ради одной иконки.

[lamer350]

Дархан Камалиев, вопрос критичный, ибо в поисковые системы будут индексировать всю кашу, человек в завернул тупо весь сайт. В поисковой выдаче потом будет просто каша в описании к сайту. В 20к19 году это важнейшая часть сайта, если не умеете пользоваться HTML5 тегами - лучше их вообще не использовать ибо будет только хуже (это обращение не конкретно к вам, Дархан, а ко всем кто читает).

[lamer350]

VegasChickiChicki, Я же писал вам конкретно что не так с фото, вы читаете мимо строк видимо) Изначально у вас размер фото оверсайз, далее смотреть нужно на разрешение, нет адаптации изображений

[VegasChickiChicki]

lamer350, Попытался изменить строение HTML документа с соблюдением семантики.
Разбил контентные части на секции , у каждой части присутствует заголовок первого уровня ,если есть необходимость ,то присутствуют подзаголовки. Добавил 2 метатега : ключевые слова и описание сайта. Хотелось бы попросить взглянуть еще раз и объяснить где стало хуже ,где лучше ,буду очень признателен.