Как можете оценить качество моего кода HTML и CSS?

Question

[holvender]

Хотелось бы попросить людей читающих этот вопрос оценить качество моего кода.

https://codepen.io/holvender/pen/poPwxWb?editors=1100

Comments

[archelon]

картинки не грузятся, адаптива нет
пока оценка 0

[holvender]

archelon, я вед уточнил, что нужна оценка кода, а не готовой верстки)

[archelon]

Holvender, а смысл смотреть на код отдельно от верстки?
уточните, что именно нужно оценить

[WapSter]

Каскадный код, в девтулзы не смотришь. Песочница и так уже имеет теги head и body. Такое себе

[holvender]

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

[Froggyweb]

Holvender,
и код такой же. хотя это не код, а разметка стилей

.works_nav ul li a:hover{
    color: #C0301C;
}

и т.д и т.п

Answer 1

[Николай Угольников]

6/10
Поработай над доступностью сайта

Comments

[holvender]

Под доступностью подразумевается адаптив?

[Николай Угольников]

Holvender, не только, у <button> указывать атрибут aria-label, у<img> указыватьalt и т.д.

[holvender]

Николай Угольников, понял, спасибо.

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

Николай Угольников, рановато ему забивать себе мозги доступностью, есть ещё что учить

Answer 2

[Даниил Секретов]

По моему код красивый! И удобный.

Answer 3

[Алексей Дубровин]

Тк как вы используете новые теги, то можно добавить
1) picture
2) nav
Всё чаще и чаще встречаются.

Тк же определитесь с методологией. Это вроде бэм но и нет.
Всё-же программисты - командные игроки. И чем легче можно понять код другим тем лучше.
Стандарты облегчают передачу в команде.

div.post_right-section div.lol div p:nth-of-type(1)

Облегчайте код. 7 вложенность это очень много(И для восприятия и для браузера.)
Сейчас конечно железо становиться всё мощнее и мощнее но всё-же.
content: url('../img/mobile_phone.png');
Все иконки переводятся обычно в svg а потом в css

Comments

[holvender]

Благодарю за замечания!