Sprint 1 #4
Sprint 1 #4
Conversation
✅ Deploy Preview for legendary-cat-a62c0e ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Здравствуйте. (Нужно развернуть общий комментарий ↓)
Посмотрите на гитхабе все комментарии к коду (нужно прокрутить вниз страницу там)
Работа проделана огромная
Что сделано отлично:
Readmeхорошо оформлен- Проект задеплоен и отлично работает
- Отлично, что не забыли про
.gitignore - Отлично, что ограничиваете версию
node - Отлично, что нет
EOFошибок в гите - Хорошая структура папок и файлов
- Отлично, что цвета вынесены в
css-константы - Отлично, что не забываете про тег
mainдля главной части страницы
Что нужно исправить:
- Нужно обернуть список с ссылками в тег
navпо семантике, так как это навигация по сайту.
Исправьте, пожалуйста, недочеты и работа будет принята. Пожалуйста, проверьте работоспособность проекта и наличие возможных ошибок в консоли браузера (кнопка F12) перед отправкой на ревью.
Напоминаю, что работа может быть принята только после исправления всех критических замечаний Нужно исправить.
Комментарии Можно лучше не обязательны к исправлению прямо сейчас, это рекомендации
Удачного рефакторинга кода.
src/pages/main-page/main-page.hbs
Outdated
| @@ -0,0 +1,12 @@ | |||
| <div class="main-page__container"> | |||
| <div class="main-page__items"> | |||
There was a problem hiding this comment.
Нужно исправить
Нужно обернуть список с ссылками в тег nav по семантике, так как это навигация по сайту.
Можно лучше
Чтобы сделать сайт более доступным для пользователя, нужно стараться придерживаться семантической верстки.
Несколько полезных статей на эту тему:
• Что такое семантическая вёрстка и зачем она нужна
• Семантика для циников
No description provided.