Sprint 1 #61
Sprint 1 #61
Conversation
There was a problem hiding this comment.
Здравствуйте. (Нужно развернуть общий комментарий ↓)
Посмотрите на гитхабе все комментарии к коду (нужно прокрутить вниз страницу там)
Работа проделана огромная
Что сделано отлично:
Readmeхорошо оформлен- Отлично, что не забыли про
.gitignore - Отлично, что ограничиваете версию
node - Отлично, что нет
EOFошибок в гите - Хорошая структура папок и файлов
- Отлично, что не забываете про
altв тегахimg
Что нужно исправить:
- По ссылке на задеплоеный проект страница недоступна
- По семантике нужно помещать весь главный контент страницы в тег
main.
Что можно улучшить:
- все цвета лучше вынести в
css-переменные и использовать их, чтобы можно было мгновенно понимать, что это за цвет, и можно было бы подкорректировать его в одном месте и не искать дублирования во всем коде
Исправьте, пожалуйста, недочеты и работа будет принята. Пожалуйста, проверьте работоспособность проекта и наличие возможных ошибок в консоли браузера (кнопка F12) перед отправкой на ревью.
Напоминаю, что работа может быть принята только после исправления всех критических замечаний Нужно исправить.
Комментарии Можно лучше не обязательны к исправлению прямо сейчас, это рекомендации
Удачного рефакторинга кода.
| - http://localhost:3000/404 | ||
| - http://localhost:3000/500 - страница ошибки | ||
|
|
||
| https://fastidious-salamander-621cd8.netlify.app |
There was a problem hiding this comment.
Нужно исправить
- По ссылке на задеплоеный проект страница недоступна
| justify-content: space-between; | ||
|
|
||
| &-value { | ||
| color: #898585; |
There was a problem hiding this comment.
Можно лучше
все цвета лучше вынести в css-переменные и использовать их, чтобы можно было мгновенно понимать, что это за цвет, и можно было бы подкорректировать его в одном месте и не искать дублирования во всем коде
| @@ -0,0 +1,3 @@ | |||
| <div class="container"> | |||
There was a problem hiding this comment.
Нужно исправить
По семантике нужно помещать весь главный контент страницы в тег main.
Чтобы сделать сайт более доступным для пользователя, нужно стараться придерживаться семантической верстки.
Несколько полезных статей на эту тему:
• Что такое семантическая вёрстка и зачем она нужна
• Семантика для циников
No description provided.