Code Review вёрстки?

Прошу строгой критики в свой адрес.
В первую очередь меня интересует качество моего кода. Допускаю ли я какие-нибудь критические (или не очень) ошибки из раза в раз?

Первый проект: okitonori.ru/portfolio/mogo/index.html (https://github.com/Okitonori/MoGo )
Второй проект: okitonori.ru/portfolio/activebox/index.html (https://github.com/Okitonori/ActiveBox )
  • Вопрос задан
  • 542 просмотра
Решения вопроса 1
sfi0zy
@sfi0zy Куратор тега Вёрстка
UI developer. Верстаю неверстаемое.
1. Картинки, как и любые бинарники, в репозиториях лучше не хранить. Любые изменения в них - и они будут раздувать размер репозитория до безобразия. Для демо - норм, но по жизни - лучше картинки хранить отдельно.
2. Почему минифицированные зависимости лежат прямо в исходниках, а не подгружаются по необходимости в node_modules?
3. Цвета и шрифты в стилях то задаются как переменные, то как магические значения в коде. Лучше уж все делать на переменных, раз уж начали.
4. Скриптов слишком мало, чтобы что-то говорить.
5. Во втором лендинге все стили в одну кучу - не удобно, лучше бы поделить как в первом. И писать префиксы руками в 2020 году - не комильфо. Для этого есть автопрефиксер.
6. Табом страницы не табаются. Ну то есть табаются, но этого местами совсем не видно, а некоторые элементы как я понимаю совсем недостижимы.
7*. Есть проблемы с контрастностью. Вероятно это косяк дизайнера, но вы, как профессионал в своей области, должны такое поправлять, даже если с самим дизайнером связи нет (как в случае с найденным в сети макетом).
Ответ написан
Пригласить эксперта
Ответы на вопрос 1
prrrrrrr
@prrrrrrr
Верстаю сразу на PHP.
В первую очередь меня интересует качество моего кода

а вы уверены что это ваш код? или вы скачали шаблон и поместили себе на хостинг?
Ответ написан
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы