Могу ли я получить code review по выполненной работе?

Question

[Егор Жолнин]

Лучшее обучение - это практика. Думая подобным образом, взял макет и начал его верстать. Сверстал, теперь хотелось бы получить feedback от людей, которым это будет интересно. Ссылка на репозиторий: https://github.com/daydarkln/testSite
Если на вопрос, можно получить ревью или нет вы захотите написать: "Можно!", то пожалуйста, поделитесь ресурсом, где это "Можно!" можно осуществить. Спасибо за внимание)

Comments

[alex]

сделайте github pages

[Алекс Глебов]

в css насчитал 6 уровней вложенности!
а если в разметке что-то поменяется? если нужно использовать блок повторно?
верстка должна быть гибкой, сайт же не картинка, он динамичен.

Answer 1

[Денис Инешин]

Есть хорошие чек-листы, по началу очень хорошо сверятся по ним:
https://gist.github.com/justinhillsjohnson/5503121
sideproject.io/front-end-code-review-goodness
https://css-tricks.com/what-a-css-code-review-migh...

От меня.
1. Не используйте reset.css, используйте normalize.css
2. Вначале подключают reset/normalize, а потом уже ваши стили.
3. Вы явно не используете нормальный редактор кода (вроде JetBrains WebStorm). У вас постоянно все по разному написано, например скобочки то прижаты к слову, то разделены пробелом и т.п.
4. В CSS у вас мешанина, хотя в начале вроде на БЭМ похоже, но нет. Если не нравится БЭМ, используйте rscss.io

Comments

[Егор Жолнин]

Я ST3 использую) Спасибо за ответ, буду прокачиваться

Answer 2

[string15]

Не благодарите)

Comments

[Егор Жолнин]

Достаточно полезная информация, если пока не шаришь в back-end'е, а в нем я не бум-бум)