Оцените сайт джуниора, какие минусы?

Question

[Данил Полянский]

Сайт: https://danilpolyanskiy.github.io/index.html

Я джуниор в веб-разработке, интересно узнать мнение других людей о этом сайте.

Comments

[Vlad Osadchyi]

Мобильная версия это просто катастрофа

[Данил Полянский]

Vlad Osadchyi, это да, знаю, не занимался пока что адаптивностью, интересно узнать именно оценку дизайна.

[Vlad Osadchyi]

Данил Полянский, сложно оценивать дизайн, когда проблемы с самой вёрсткой )

[Данил Полянский]

Vlad Osadchyi, ну проблемы вроде как только с маленькими экранами смартфонов. На мониторах от ноутбука или ПК картинка будет вполне нормальной для всех размеров.

[Vlad Osadchyi]

Данил Полянский, ну я с телефона )

[WapSter]

Данил Полянский, просто трех этажным матом вас бы крыть. Нафига каждый вот такой выкладывает оценить, что-нибудь и не доделывает это "что-нибудь". Нет мобильности это вообще катастрофический не до дел, в мире где 80% мобильный траффик.

[Данил Полянский]

WapSter, меньше агрессии, этот сайт я написал и отправил сюда на оценку, чтобы понять что я сделал не так, и исправить это, тем самым узнав больше и прокачав свои умения в верстке сайта.

P.S. Чтобы делать адаптивную версию сайта, надо с компьютерной до конца разобратся, а тут я с компьютерной не разобрался до конца а вы уже хотите адаптив.

[DmitrijP]

Данил Полянский, делайте mobile first, на данный момент ваш сайт - полное дерьмо.

[Antonio Solo]

надо еще что-то оценивать ?

[IvanSafronov91]

Мне вот любопытно, а кто назвал тебя джуном? Ты сам?

[Данил Полянский]

Интересно кто тебя вообще в жизни держит до сих пор с таким образом жизни токсичным. Был бы ты в 90-х вышел бы на улицу - домой живым не вернулся.

[IvanSafronov91]

Данил Полянский, и мне снова любопытно, а кого ты токсичным называешь? Меня? Ни разу тебя не оскорбившим? На свои то комментарии взглянуть не хочешь, оценить их?

Так кто тебя джуном то назвал?)

[Данил Полянский]

IvanSafronov91, ты бредишь.

[IvanSafronov91]

Данил Полянский, джуном кто назвал? Сколько еще раз спросить?

[php7788]

IvanSafronov91, вот дурак ты. Жаль что такие люди есть в мире...

Answer 1

[xdevelx]

1. Теги h1 и h2 нужно использовать для заголовков, а не для всего подряд.
2. В теге ul должны быть элементы списка, а не ссылки. https://developer.mozilla.org/ru/docs/Web/HTML/Ele...
3. В img должен быть атрибут alt (для декоративных - пустой)
4. Разметку нужно проверять валидатором https://validator.w3.org/
5. Иконки нужно не в png,а svg (и в спрайт объединить)
6. Ширину, да и высоту кнопки не стоит делать фиксированными (по крайней мере min-width)
7. Контейнеры разной ширины предусмотрены дизайном?
8. letter-spacing: -0.4; -0.4 - это сколько? https://developer.mozilla.org/ru/docs/Web/CSS/lett...
9. Ссылкам в меню задайте padding - пользователь не должен целиться в область ссылки
10. Скрипты из head перенести перед </body> либо атрибут async


chrome dev tools => audits => accessibility страдает явно

Comments

[Одиночка Айс]

В img должен быть атрибут alt (для декоративных - пустой)

он в принципе для всех пустой должен быть, ибо никто этим не занимается.

Answer 2

[Антон Литвиненко]

А ты прогони его через валидатор и сам все увидишь)
Адаптивки нет, ну ок.
Скрипты подключать в подвал.
В разметке ошибки, даже хром не смог распарсить.
http: //prntscr.com/rdat6p
Инлайн стили, повторяющиеся заголовки первого уровня, отсутствие alt у img, отсутствие семантики.
Скрипт прокрутки по анкорам вызывает ошибки, потому что неверно задан таргет.
Это то что за пару минут в глаза попало