Sprint 1 #31
Sprint 1 #31
Conversation
There was a problem hiding this comment.
Здравствуйте. (Нужно развернуть общий комментарий ↓)
Работа проделана огромная
Что сделано отлично:
Readmeхорошо оформлен- Проект задеплоен и отлично работает
- Отлично, что не забыли про
.gitignore - Отлично, что ограничиваете версию
node - Отлично, что нет
EOFошибок в гите - Хорошая структура папок и файлов
- Отлично, что цвета вынесены в
css-константы - Отлично, что не забываете про
altв тегахimg - Отлично, что моковые данные вынесены в файл конфигурации
- Отлично, что установили язык страницы
Что нужно исправить:
- По семантике нужно помещать весь главный контент страницы в тег
main. - Нужно обернуть кнопки навигации в тег
navпо семантике, так как это навигация по сайту.
Что можно улучшить:
- Нужно ещё указать
viewportдля отзывчивости страницы
Исправьте, пожалуйста, недочеты и работа будет принята. Пожалуйста, проверьте работоспособность проекта и наличие возможных ошибок в консоли браузера (кнопка F12) перед отправкой на ревью.
Напоминаю, что работа может быть принята только после исправления всех критических замечаний Нужно исправить.
Комментарии Можно лучше не обязательны к исправлению прямо сейчас, это рекомендации
Удачного рефакторинга кода.
| <script src="main.js" type="module"></script> | ||
| </head> | ||
| <body> | ||
| <div id="app" /> |
There was a problem hiding this comment.
Нужно исправить
По семантике нужно помещать весь главный контент страницы в тег main.
Чтобы сделать сайт более доступным для пользователя, нужно стараться придерживаться семантической верстки.
Несколько полезных статей на эту тему:
• Что такое семантическая вёрстка и зачем она нужна
• Семантика для циников
src/pages/chat/chat.hbs
Outdated
| <section> | ||
| <div class="content"> | ||
| <span>Выберите чат чтобы отправить сообщение</span> | ||
| <div class="list__pages"> |
There was a problem hiding this comment.
Нужно исправить
Нужно обернуть кнопки навигации в тег nav по семантике, так как это навигация по сайту.
Чтобы сделать сайт более доступным для пользователя, нужно стараться придерживаться семантической верстки.
Несколько полезных статей на эту тему:
• Что такое семантическая вёрстка и зачем она нужна
• Семантика для циников
| <meta charset="UTF-8"> | ||
| <title>Middle Messenger Prakticum Yande</title> | ||
| <script src="main.js" type="module"></script> | ||
| </head> |
There was a problem hiding this comment.
Можно лучше
Нужно ещё указать viewport для отзывчивости страницы
<meta name="viewport" content="width=device-width, initial-scale=1, viewport-fit=cover">
Так же есть еще ряд рекомендаций для meta, которые доступны тут
No description provided.